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.

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

Update all dependencies #36

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Update all dependencies #36

wants to merge 2 commits into from

Conversation

zackw
Copy link

@zackw zackw commented Dec 2, 2022

Update all dependencies to their latest .0 releases (except for cheerio, which is still working on its 1.0.0 -- I wouldn't normally write a >= dependency on a release candidate but the goal here is to get rid of the transitive dependency on an old version of css-select, and cheerio only got around to fixing that in the middle of their release candidate series).

Subsumes dependabot PRs #27, #28, #30, #34, and #35.

Lockfiles shouldn’t exist at all, but most of the harm they do can be
mitigated by keeping them as ephemeral as possible.  In particular,
they should not be committed to version control ever.  Delete the
erroneously committed lockfile and discourage it from being re-added
by adding it to .gitignore.  /En passant/ add leading slashes to
several other entries in .gitignore that should only exist at top level.

This, all by itself, resolves 12 of the 17 buggy dependencies reported
by ‘npm audit‘, neatly demonstrating why lockfiles are bad.
This is minor except for cheerio, which goes from 0.22.0 to
1.0.0~rc.12, and mocha, which goes from 6.x to 10.x.  No code
changes are required, all tests pass.

Bump version to 1.0.3.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant