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

github: Add dependabot configuration #81812

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

pdgendt
Copy link
Collaborator

@pdgendt pdgendt commented Nov 23, 2024

Add a GH dependabot configuration to check for updates for github actions and pip dependencies once a week.

Add a GH dependabot configuration to check for updates for github
actions and pip dependencies once a week.

Signed-off-by: Pieter De Gendt <[email protected]>
Copy link
Member

@stephanosio stephanosio left a comment

Choose a reason for hiding this comment

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

I do not see much point in enabling Dependabot right now when we do not currently version-lock packages (we should look into doing that as previously discussed ...).

Copy link
Collaborator

@kartben kartben left a comment

Choose a reason for hiding this comment

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

Thanks for finally sending the PR I kept delaying forever :)

I had done it slightly differently for the sake of having slightly better PR titles with proper prefixes based on the area. Can't recall why I didn't put any label (it might be that the dependabot "user" doesn't have permission to set them? Really don't remember, sorry) but that would be a nice addition too?
6b14bea

@kartben
Copy link
Collaborator

kartben commented Nov 23, 2024

I do not see much point in enabling Dependabot right now when we do not currently version-lock packages (we should look into doing that as previously discussed ...).

Yep but it is useful for GH actions nevertheless, as this can let us know when to bump the not-fully-locked-but-still-somewhat-pinned versions.

@pdgendt
Copy link
Collaborator Author

pdgendt commented Nov 23, 2024

Thanks for finally sending the PR I kept delaying forever :)

I had done it slightly differently for the sake of having slightly better PR titles with proper prefixes based on the area. Can't recall why I didn't put any label (it might be that the dependabot "user" doesn't have permission to set them? Really don't remember, sorry) but that would be a nice addition too? 6b14bea

I like yours better! The default dependabot labels aren't really useful so we should let zephyr bot do that part.

EDIT: we should however group the packages into single PRs, instead of a PR for each dependency IMO.

@pdgendt
Copy link
Collaborator Author

pdgendt commented Nov 23, 2024

I do not see much point in enabling Dependabot right now when we do not currently version-lock packages (we should look into doing that as previously discussed ...).

Yes, we can simply set fixed versions in the requirements files. This would increase reproducibility and prevent packages break our users' environments.

@henrikbrixandersen
Copy link
Member

I do not see much point in enabling Dependabot right now when we do not currently version-lock packages (we should look into doing that as previously discussed ...).

Yes, we can simply set fixed versions in the requirements files. This would increase reproducibility and prevent packages break our users' environments.

Fixed version would also likely introduce version incompatibles with other requirements (e.g. for downstream tooling). It also means more work on maintaining release branches, ensuring dependencies are updated in all active branches as well. We have discussed this in the past and opted not to lock on versions unless really needed.

@rettichschnidi
Copy link
Contributor

rettichschnidi commented Nov 23, 2024

Fixed version would also likely introduce version incompatibles with other requirements (e.g. for downstream tooling). It also means more work on maintaining release branches, ensuring dependencies are updated in all active branches as well. We have discussed this in the past and opted not to lock on versions unless really needed.

How about something like lock files? Having a pretty lose description of minimum versions, but then adding a refreshed lock file on each release? People can still ignore/update it, but one would be able to restore a "same-versions-as-back-then" Zephyr build environment many years in the future.

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.

5 participants