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 #225

Merged
merged 7 commits into from
Jan 6, 2025
Merged

Conversation

bdice
Copy link
Contributor

@bdice bdice commented Jan 4, 2025

Combines changes from several PRs, to reduce CI churn. Closes #221, closes #222, closes #223, closes #224.

  • Update mikefarah/yq Docker tag to v4.44.6
  • Update dependency cli/cli to v2.64.0
  • Update dependency codecov-cli to v9.1.1
  • Update amazon/aws-cli Docker tag to v2.22.28

@bdice bdice requested a review from a team as a code owner January 4, 2025 17:45
@bdice bdice requested review from KyleFromNVIDIA and removed request for a team January 4, 2025 17:45
Copy link

copy-pr-bot bot commented Jan 4, 2025

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@bdice
Copy link
Contributor Author

bdice commented Jan 4, 2025

/ok to test

@jameslamb jameslamb self-requested a review January 6, 2025 14:24
Copy link
Member

@jameslamb jameslamb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Agreed!

I'd support grouping these specific updates (i.e. NOT conda) together so dependabot generates grouped PRs like this, if you want: https://docs.github.com/en/code-security/dependabot/working-with-dependabot/dependabot-options-reference#groups--

@jameslamb jameslamb removed the request for review from KyleFromNVIDIA January 6, 2025 14:26
@jameslamb
Copy link
Member

Sorry, forgot this is renovate not dependabot.

But renovate has the same "group multiple dependencies into 1 PR" functionality: https://docs.renovatebot.com/noise-reduction/#package-grouping

@bdice
Copy link
Contributor Author

bdice commented Jan 6, 2025

@jameslamb That grouping would be a welcome change. Want to make a PR to group everything?

@bdice bdice merged commit a1fd1fb into rapidsai:main Jan 6, 2025
407 checks passed
@jameslamb
Copy link
Member

Ok great, yep I can do that.

@jameslamb
Copy link
Member

Tried for a bit today and couldn't quite figure out how to make grouping work together with our use of regex matching:

"regexManagers": [

Opened an issue to track doing that, will come back to it at some point in the future: #226

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