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

Chore: Updating Dependencies #267

Merged
merged 2 commits into from
Aug 6, 2024
Merged

Conversation

karnthis
Copy link
Contributor

This PR combines Dependabot PRs #244 #260 #261 #263 #264 and #265 and includes some additional audit fixes Dependabot missed. Build succeeded, Tests pass.

Please update the Dependabot settings so I stop getting roughly 30 emails from this project every Monday evening.

@karnthis
Copy link
Contributor Author

karnthis commented Aug 4, 2024

bump for @Rycochet to see

@Rycochet
Copy link
Collaborator

Rycochet commented Aug 6, 2024

I'm just back from 2 weeks of busy - will approve this, but if you have the time (before I do) - there's a code as config that can tell it to update things as groups of updates together - like testing, linting, building etc - it's on my TODO list and I'll try to have a look next week if you don't

@Rycochet Rycochet merged commit 7d6da15 into pieroxy:master Aug 6, 2024
6 of 7 checks passed
@karnthis
Copy link
Contributor Author

karnthis commented Aug 6, 2024

I'm just back from 2 weeks of busy - will approve this, but if you have the time (before I do) - there's a code as config that can tell it to update things as groups of updates together - like testing, linting, building etc - it's on my TODO list and I'll try to have a look next week if you don't

Sorry not sure what you're referring to. Can you elaborate?

@Rycochet
Copy link
Collaborator

Rycochet commented Aug 6, 2024

https://docs.github.com/en/code-security/dependabot/dependabot-version-updates/configuration-options-for-the-dependabot.yml-file#groups - hopefully that page makes things more clear - it should be able to cut down the number of PRs generated 😄

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.

2 participants