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 dependabot.yml with new PR groups for eslint, testing dependencies, postcss, sass, webpack #3482

Merged
merged 1 commit into from
Oct 18, 2024

Conversation

tdonohue
Copy link
Member

@tdonohue tdonohue commented Oct 18, 2024

Description

Small PR to update our Dependabot rules with new groups for many major dependencies including:

  • ESLint
  • Testing (Jasmine, Karma, Cypress, etc)
  • PostCss
  • Sass
  • Webpack

These same groups have been enabled for main, dspace-8_x and dspace-7_x

The goal is just to minimize the number of PRs created by Dependabot by ensuring updates to these dependencies are grouped into a single PR. Often, this is the best approach, as when you update one dependency in this group, you usually have to update them all.

This PR is just to document the reasons for this change. The only way to test it is to merge it. Merging immediately.

@tdonohue tdonohue added code task 1 APPROVAL pull request only requires a single approval to merge labels Oct 18, 2024
@tdonohue tdonohue added this to the 9.0 milestone Oct 18, 2024
@tdonohue tdonohue merged commit 1592763 into main Oct 18, 2024
9 checks passed
@tdonohue tdonohue deleted the tdonohue-patch-1 branch October 18, 2024 20:15
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 code task
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

1 participant