Skip to content

Bump all @typescript-eslint/* dependencies to 7.18.0#3457

Merged
tdonohue merged 2 commits intoDSpace:mainfrom
tdonohue:update_all_eslint
Oct 15, 2024
Merged

Bump all @typescript-eslint/* dependencies to 7.18.0#3457
tdonohue merged 2 commits intoDSpace:mainfrom
tdonohue:update_all_eslint

Conversation

@tdonohue
Copy link
Copy Markdown
Member

@tdonohue tdonohue commented Oct 15, 2024

Description

Follow-up to #3453 from dependabot (which was merged earlier today)

After merging this PR, I discovered that all @typescript-eslint/* dependencies need to be updated together. This PR therefore updates the other @typescript-eslint/* dependencies to the same version (7.18.0), and adds a new rule to dependabot.yml to ensure they are updated together in the future.

Updated all together by running:

npx npm-check-updates --upgrade --target "minor" --filter "/@typescript-eslint/"

See https://stackoverflow.com/a/67583172/3750035

Instructions for Reviewers

@tdonohue tdonohue added dependencies Pull requests that update a dependency file 1 APPROVAL pull request only requires a single approval to merge labels Oct 15, 2024
@tdonohue tdonohue added this to the 9.0 milestone Oct 15, 2024
@tdonohue tdonohue changed the title Update all @typescript-eslint dependencies to 7.18.0 Bump all @typescript-eslint dependencies to 7.18.0 Oct 15, 2024
@tdonohue tdonohue changed the title Bump all @typescript-eslint dependencies to 7.18.0 Bump all @typescript-eslint/* dependencies to 7.18.0 Oct 15, 2024
@tdonohue
Copy link
Copy Markdown
Member Author

Merging immediately, as this fixes the broken lint tests on main.

@tdonohue tdonohue merged commit a1fcc2e into DSpace:main Oct 15, 2024
@tdonohue tdonohue deleted the update_all_eslint branch October 15, 2024 22:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

1 APPROVAL pull request only requires a single approval to merge dependencies Pull requests that update a dependency file

Projects

No open projects
Status: ✅ Done

Development

Successfully merging this pull request may close these issues.

1 participant