Skip to content

ci: enhance dependabot config to track pip and cargo packages#658

Open
lhoupert wants to merge 3 commits intomainfrom
ci--enhance-dependabot-config-to-track-pip-and-cargo-package
Open

ci: enhance dependabot config to track pip and cargo packages#658
lhoupert wants to merge 3 commits intomainfrom
ci--enhance-dependabot-config-to-track-pip-and-cargo-package

Conversation

@lhoupert
Copy link
Copy Markdown
Contributor

@lhoupert lhoupert commented Apr 3, 2026

Follow up from #647. Adding pip and cargo package to dependabot that should help quickly patch vulnerabilities in dependencies package

@lhoupert lhoupert requested a review from kylebarron April 3, 2026 22:24
@github-actions github-actions bot added the ci label Apr 3, 2026
@lhoupert
Copy link
Copy Markdown
Contributor Author

lhoupert commented Apr 3, 2026

@kylebarron I am not sure what "/pyo3-bytes" and "/pyo3-object-stores" are used for and if they should also be bumped in the same way

- "minor"
- "patch"

- package-ecosystem: "uv"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

What's the difference between package ecosystem uv and pip?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Dependabot introduce uv support last year. The main difference is that pip tracks pyproject.toml but ignores uv.lock, leaving the lockfile out of sync after each Dependabot PR. Using uv tells Dependabot to use uv's own resolver and keep uv.lock in sync automatically.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Do you have a preference on which one to use @kylebarron ?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I think uv is always correct then, or else the uv.lock will be out of sync. I only use uv in all my projects

Comment on lines +59 to +85
- package-ecosystem: "uv"
directory: "/"
schedule:
interval: "weekly"
day: "monday"
groups:
minor-and-patch:
applies-to: version-updates
patterns:
- "*"
update-types:
- "minor"
- "patch"

- package-ecosystem: "pip"
directory: "/obstore"
schedule:
interval: "weekly"
day: "monday"
groups:
minor-and-patch:
applies-to: version-updates
patterns:
- "*"
update-types:
- "minor"
- "patch"
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Similar to developmentseed/lonboard#1162 (comment), do we need dependabot for python, where it's not forming part of a distributed application

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

🤷‍♂️

Removed Cargo package ecosystems for pyo3-object_store and pyo3-bytes.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants