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 dependencies #228

Merged
merged 7 commits into from Sep 24, 2021
Merged

Update dependencies #228

merged 7 commits into from Sep 24, 2021

Conversation

adalinesimonian
Copy link
Member

@adalinesimonian adalinesimonian commented Sep 24, 2021

Updates most dependencies, with two caveats:

  • stylelint is updated in the lockfile to 13.7.2, but not yet to the current version.

    The changes from Report disables in the same manner as lints stylelint#4973, released in 13.8.0, will require some more significant changes. The extension is expecting the old disable reporting format, and updating the dependency breaks tests. I think it would be best to have that update as a separate PR. At the moment, the proposed Update stylelint to 13.13.1 #205 only updates the entry in package.json, and Bump stylelint from 13.6.0 to 13.13.1 #197, being an automated PR, also doesn't address the changes.

  • p-wait-for is updated to its latest 3.x version but not to the current 4.x.

    p-wait-for 4.x has been converted to a native ESM package, and due to limitations of testing in VS code, dynamic imports or other attempts to get this package to work in workspace tests won't work quite yet. In the meantime, Dependabot has been configured to ignore major version updates for p-wait-for.

@adalinesimonian adalinesimonian self-assigned this Sep 24, 2021
@adalinesimonian adalinesimonian merged commit 439c102 into master Sep 24, 2021
@adalinesimonian adalinesimonian deleted the upgrade-deps branch September 24, 2021 00:36
@adalinesimonian adalinesimonian added this to the v0.87.0 milestone Sep 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr: dependencies relates to dependencies
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant