-
Notifications
You must be signed in to change notification settings - Fork 83
Description
The current default value for cache-dependency-glob is not flexible enough to be used by most people.
cache-dependency-glob: |
**/requirements*.txt
**/uv.lockA more generic value like below should cover over 99.5% of use cases:
cache-dependency-glob: |
**/*(requirements|constraints)*.(txt|in)
**/pyproject.toml
**/uv.lockReasoning:
- There is no clear winner in battle between sort-friendly and english-syntax:
test-requirements.txtvsrequirements-test.txt, so supporting only one will affect ~50% users constraints.txtis very popular against those using constraint files- lots of users are using
.ininstead of.txtextension because this is friendly with tools like dependabot, which can cause problems when mixing lock files with unlocked ones..inis a way of saying that the file is clearly not a lock file. pyproject.tomlshould not need any explanation- Not having a set of defaults that work for most users makes it harder to reuse the action, especially by other reusable actions/workflows, as users will need to find a way to pass customizations to the reused action. With a generic enough default, the need to it disappears.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels