Skip to content

Update all dependencies #83

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

Merged
merged 1 commit into from
Jan 28, 2025
Merged

Update all dependencies #83

merged 1 commit into from
Jan 28, 2025

Conversation

renovate[bot]
Copy link
Contributor

@renovate renovate bot commented Jan 23, 2025

This PR contains the following updates:

Package Type Update Change
actions/cache action major v3 -> v4
actions/checkout action major v3 -> v4
bazel_skylib bazel_dep minor 1.5.0 -> 1.7.1
rules_python bazel_dep major 0.31.0 -> 1.1.0

Release Notes

actions/cache (actions/cache)

v4

Compare Source

actions/checkout (actions/checkout)

v4

Compare Source


Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.

👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.


  • If you want to rebase/retry this PR, check this box

This PR was generated by Mend Renovate. View the repository job log.


This change is Reviewable

@jwnimmer-tri
Copy link
Contributor

jwnimmer-tri commented Jan 23, 2025

+@rpoyner-tri for both reviews, please.

This introduces automated dependency upgrades for RobotLocomotion/models, like we already have for RobotLocomotion/drake-blender.

Also please review this file already landed on master (which I needed to merge ahead of time in order to get the bot to run):
https://github.com/RobotLocomotion/models/blob/master/.github/renovate.json

Docs are at https://docs.renovatebot.com/presets-config/#configrecommended and nearby. Not sure whether / where I should put that in a code comment somewhere?

Copy link
Contributor

@rpoyner-tri rpoyner-tri left a comment

Choose a reason for hiding this comment

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

maybe document it in robots.txt ;) ?

Srsly, there should probably be a markdown file somewhere that at least has pointers to significant maintenance practices. Not sure what it should be called.

Reviewable status: all discussions resolved, LGTM missing from assignee rpoyner-tri, platform LGTM missing

Copy link
Contributor

@rpoyner-tri rpoyner-tri left a comment

Choose a reason for hiding this comment

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

Reviewed 2 of 2 files at r1, all commit messages.
Reviewable status: all discussions resolved, LGTM missing from assignee rpoyner-tri, platform LGTM missing (waiting on @renovate[bot])

Copy link
Contributor

@rpoyner-tri rpoyner-tri left a comment

Choose a reason for hiding this comment

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

:lgtm: immediate changes look fine. I'll let you decide what to do about docs.

Reviewable status: :shipit: complete! all discussions resolved, platform LGTM from [rpoyner-tri] (waiting on @renovate[bot])

Copy link
Contributor

@rpoyner-tri rpoyner-tri left a comment

Choose a reason for hiding this comment

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

Reviewable status: 1 unresolved discussion, platform LGTM from [rpoyner-tri] (waiting on @renovate[bot])


a discussion (no related file):
nit placeholder discussion about maybe more docs, to suppress the merge button a bit longer.

Copy link
Contributor

@jwnimmer-tri jwnimmer-tri left a comment

Choose a reason for hiding this comment

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

Reviewed 2 of 2 files at r1, all commit messages.
Reviewable status: :shipit: complete! all discussions resolved, platform LGTM from [rpoyner-tri] (waiting on @renovate[bot])


a discussion (no related file):

Previously, rpoyner-tri (Rick Poyner (rico)) wrote…

nit placeholder discussion about maybe more docs, to suppress the merge button a bit longer.

=> #84

@jwnimmer-tri jwnimmer-tri merged commit 450fe9b into master Jan 28, 2025
2 checks passed
@jwnimmer-tri jwnimmer-tri deleted the renovate/all branch January 28, 2025 17:47
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