Upgrade gradle/actions to v6 with basic caching#23
Conversation
Use the MIT-licensed basic caching provider (a thin wrapper over actions/cache) instead of the default enhanced caching, which is proprietary and subject to separate terms of use. Resolves ENG-4638 Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
These are used by precious for pre-commit linting. Previously prettier was invoked via npx, requiring Node.js to be installed separately. Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the .precious.toml configuration to call prettier directly and simplifies the stderr ignore list. It also adds yamllint and prettier to the mise.toml configuration. A review comment points out that these tools may not be available in the default mise registry and suggests using explicit provider prefixes like pip: and npm: to ensure successful installation.
| yamllint = "latest" | ||
| # prettier is used by precious for Markdown formatting | ||
| prettier = "latest" |
There was a problem hiding this comment.
The tools yamllint and prettier are not included in the default mise registry. To ensure they are correctly installed via mise install as intended by the PR description, you should use explicit provider prefixes. Using pip: for yamllint and npm: for prettier is the standard approach. Note that these providers require the corresponding runtimes (Python and Node.js) to be available on the system or managed via mise.
"pip:yamllint" = "latest"
# prettier is used by precious for Markdown formatting
"npm:prettier" = "latest"
2 participants
Summary
gradle/actions/setup-gradlefrom v5.0.2 to v6.1.0 across all workflowsbasiccaching provider (a thin wrapper overactions/cache) instead of the defaultenhancedcaching, which is proprietary and subject to the Gradle Technologies Terms of Useyamllintandprettiertomise.tomlso all precious pre-commit dependencies are available viamise installResolves ENG-4638
Test plan
mise install && precious lint -asucceeds in a clean checkout🤖 Generated with Claude Code