Conversation
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
There was a problem hiding this comment.
Pull request overview
This PR updates the copier base template to pull in upstream changes from the copier-base-template repository, addressing yarn GPG key issues and updating various dependencies. The changes include a fix for the yarn GPG problem mentioned in the PR title, updates to development tooling versions, and new Copier tasks for automated configuration management.
Changes:
- Updated copier from 9.11.2 to 9.11.3 and ty from 0.0.13 to 0.0.17, along with numerous other dependency version bumps
- Added a Copier task to dynamically update ruff.toml target-version based on python_version, removing the manual TODO comment
- Added a Copier task to migrate .coveragerc files from repo root to backend folder, addressing issue #116
- Fixed yarn GPG key issues by adding key import to the Dockerfile
- Improved GitHub Actions workflow failure detection logic to check for success/skipped pattern instead of failure/cancelled
- Updated devcontainer features and VS Code extensions to newer versions
- Added Claude CLI integration support with ANTHROPIC_API_KEY environment variable
Reviewed changes
Copilot reviewed 29 out of 32 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
| uv.lock | Updated lock file with copier 9.11.3 and ty 0.0.17 versions |
| pyproject.toml | Updated copier and ty dependency versions |
| extensions/context.py | Updated version strings for ~40 dependencies including Python, Node.js, and various libraries |
| copier.yml | Added two new Copier tasks: one for ruff.toml target-version automation and one for .coveragerc migration |
| template/ruff.toml, ruff.toml | Removed TODO comments as ruff target-version is now managed by Copier task |
| template/.pre-commit-config.yaml, .pre-commit-config.yaml | Updated pre-commit hook versions (typos, prettier, check-jsonschema, ruff) |
| template/.github/workflows/ci.yaml.jinja, .github/workflows/ci.yaml | Improved job failure detection logic using success pattern matching |
| template/.github/workflows/pre-commit.yaml, .github/workflows/pre-commit.yaml | Updated checkout action from v6.0.1 to v6.0.2 |
| template/.github/workflows/get-values.yaml, .github/workflows/get-values.yaml | Updated checkout action from v6.0.1 to v6.0.2 |
| .github/reusable_workflows/build-docker-image.yaml | Updated checkout action from v6.0.1 to v6.0.2 |
| template/.github/actions/update-devcontainer-hash/action.yml, .github/actions/update-devcontainer-hash/action.yml | Updated checkout action from v6.0.1 to v6.0.2 |
| template/.github/pull_request_template.md, .github/pull_request_template.md | Fixed formatting by removing leading spaces before section headings |
| template/.devcontainer/devcontainer.json.jinja, .devcontainer/devcontainer.json | Updated Python feature to 1.8.0, Node feature to 1.7.1, added Claude CLI support, updated VS Code extensions |
| template/.devcontainer/Dockerfile, .devcontainer/Dockerfile | Updated base image to 5.1.4-noble, added yarn GPG key import to fix GPG issues |
| .devcontainer/install-ci-tooling.py | Updated UV_VERSION, PNPM_VERSION, and COPIER_VERSION constants |
| .devcontainer/docker-compose.yml | Added ANTHROPIC_API_KEY environment variable |
| .copier-answers.yml | Updated commit hash and install_claude_cli setting, updated Python CI version |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| exit 1 | ||
| fi | ||
| echo "✅ All jobs completed successfully or were skipped" | ||
| echo "✅ All jobs finished with skipped or success"" |
There was a problem hiding this comment.
There's a typo in this echo statement - it has an extra double-quote at the end. The closing quote should be removed so it reads: echo "✅ All jobs finished with skipped or success"
| echo "✅ All jobs finished with skipped or success"" | |
| echo "✅ All jobs finished with skipped or success" |
| "version": "{% endraw %}{{ node_version }}{% raw %}", | ||
| "pnpmVersion": "{% endraw %}{{ pnpm_version }}{% raw %}" | ||
| }{% endraw %}{% endif %}{% raw %}{% endraw %}{% if install_claude_cli %}{% raw %}, | ||
| // https://github.com/anthropics/devcontainer-features/blob/main/src/claude-code/devcontainer-feature.json |
There was a problem hiding this comment.
This line has inconsistent indentation. It should be indented with 4 spaces like the other feature entries (line 24), not 6 spaces. The comment on line 29 and the feature entry on line 30 should both use 4-space indentation to align with the other features in the JSON object.
| // https://github.com/anthropics/devcontainer-features/blob/main/src/claude-code/devcontainer-feature.json | |
| // https://github.com/anthropics/devcontainer-features/blob/main/src/claude-code/devcontainer-feature.json |
Why is this change necessary?
LabAutomationAndScreening/copier-base-template#129
How does this change address the issue?
Pulls in upstream template changes
What side effects does this change have?
N/A
How is this change tested?
Building codespace in downstream repo
Other
fixes #116
newer version of ruff allowed some simplification of code