Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Standardize test config, upgrade (some) devDeps, add named export #70

Merged
merged 11 commits into from
Feb 10, 2022
Merged

Standardize test config, upgrade (some) devDeps, add named export #70

merged 11 commits into from
Feb 10, 2022

Commits on Feb 10, 2022

  1. (refactor): move tests to test/, test-utils/ to test/config/ 

    - have been putting tests separately from source nowadays in other of
  my libraries, so standardize around that

- put fixtures just in test/ (later could be in test/fixtures/), leave
  config/ solely for test config type things
    @agilgur5
    agilgur5 committed Feb 10, 2022
    Configuration menu
    Copy the full SHA
    3748ff0 View commit details
    Browse the repository at this point in the history

  2. (env/refactor): standardize .gitignore 

    - basically the same one I use in all my other libraries
  - I believe it's an old version of Node output from gitignore.io iirc
- it has comments unlike the old one, so that's much nicer for those
  who aren't totally sure what each entry is for
    @agilgur5
    agilgur5 committed Feb 10, 2022
    Configuration menu
    Copy the full SHA
    fed1e6a View commit details
    Browse the repository at this point in the history

  3. (deps): use jest-without-globals to import jest globals 

    - no more globals used in tests!
    @agilgur5
    agilgur5 committed Feb 10, 2022
    Configuration menu
    Copy the full SHA
    c4a9ec8 View commit details
    Browse the repository at this point in the history

  4. (ci): upgrade to use Node v10 

    - Node 8 was EOL in December, I upgraded locally a while ago too

- The default NPM version for Node v10 also supports `npm ci`,
  which should be a CI speed boost as well
    @agilgur5
    agilgur5 committed Feb 10, 2022
    Configuration menu
    Copy the full SHA
    651c5c5 View commit details
    Browse the repository at this point in the history

  5. (deps): upgrade to canvas@2, jsdom@15 

    - because canvas@1 is deprecated and is having install issues
  - see hhttps://travis-ci.org/agilgur5/react-signature-canvas/builds/650962136,
    https://travis-ci.org/agilgur5/react-signature-canvas/builds/658062587,
    etc
    @agilgur5
    agilgur5 committed Feb 10, 2022
    Configuration menu
    Copy the full SHA
    754e0b6 View commit details
    Browse the repository at this point in the history

  6. (pkg): add funding

    @agilgur5
    agilgur5 committed Feb 10, 2022
    Configuration menu
    Copy the full SHA
    3962613 View commit details
    Browse the repository at this point in the history

  7. (feat): add a named export (same as default) 

    - because many folks prefer not to use default exports and because
  named exports are more explicit, etc, etc
    @agilgur5
    agilgur5 committed Feb 10, 2022
    Configuration menu
    Copy the full SHA
    0e423d4 View commit details
    Browse the repository at this point in the history

  8. (env): rename 'dist' script to 'build' 

    - 'build' is a lot more intuitive and matches other usage
    @agilgur5
    agilgur5 committed Feb 10, 2022
    Configuration menu
    Copy the full SHA
    02141b7 View commit details
    Browse the repository at this point in the history

  9. (deps): move the window.resizeTo polyfill to its own package 

    - and import & use it here!
    @agilgur5
    agilgur5 committed Feb 10, 2022
    Configuration menu
    Copy the full SHA
    e8b0554 View commit details
    Browse the repository at this point in the history

  10. env: switch to main branch as default 

    - now that GH officially supports branch renames, moving all my repos
  to use `main`
  - GH also creates redirects now, which seem to be working well for
    the badges right now, but better to move all of them anyway
  - also had to switch Codecov default branch etc
    @agilgur5
    agilgur5 committed Feb 10, 2022
    Configuration menu
    Copy the full SHA
    0c55ee1 View commit details
    Browse the repository at this point in the history

  11. ci: use travis-ci.com as .org is decomm'd 

    - https://blog.travis-ci.com/2021-05-07-orgshutdown

- made the switch over with my last PR, but needed to change the docs
  too
  - update badge to use `.com`
  - update link to use `.com`, `app.` subdomain, and `github/` URL
    that travis-ci.com currently uses

- also use `/branches` URL as the default URL shows current builds,
  which includes WIP PR/branch builds too
  - `/branches` shows the "default branch" first instead and various
    checkmarks etc, so it's a good summary page that can be drilled
    down from
    @agilgur5
    agilgur5 committed Feb 10, 2022
    Configuration menu
    Copy the full SHA
    fceda76 View commit details
    Browse the repository at this point in the history