Skip to content

Copier update (fix get values CI job)#72

Merged
ejfine merged 3 commits intomainfrom
fix-get-values
Oct 1, 2025
Merged

Copier update (fix get values CI job)#72
ejfine merged 3 commits intomainfrom
fix-get-values

Conversation

@ejfine
Copy link
Contributor

@ejfine ejfine commented Oct 1, 2025

Pull in upstream template changes

Summary by CodeRabbit

  • Chores

    • Updated multiple dependency versions across backend and frontend stacks.
    • Upgraded CI tooling version and synchronized template/context settings.
  • CI/CD

    • Improved workflow logs by pretty-printing GitHub context JSON.
  • Tests

    • Added a safeguard to prevent autofixes that could break pytest.raises contexts.
  • Developer Experience

    • Minor Dev Container comment update.
    • Formatting and comment cleanups in lint configurations with no behavioral changes.

@ejfine ejfine requested a review from Copilot October 1, 2025 11:03
@ejfine ejfine self-assigned this Oct 1, 2025
@coderabbitai
Copy link

coderabbitai bot commented Oct 1, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

This PR updates version pins and metadata across configs, tweaks a GitHub Actions step to pretty-print JSON with jq, adds Ruff unfixable SIM117 entries, and makes minor formatting/comment adjustments. No functional application logic or public interfaces change.

Changes

Cohort / File(s) Summary
Copier template metadata
.copier-answers.yml
Bump _commit from v0.0.73 to v0.0.74.
Devcontainer config
.devcontainer/devcontainer.json, .devcontainer/install-ci-tooling.py
Comment-only context hash update; bump UV_VERSION from "0.8.19" to "0.8.22".
GitHub workflow output formatting
.github/workflows/get-values.yaml, template/.github/workflows/get-values.yaml
Replace single-line echo of toJSON(github) with a here-doc piped to jq for pretty-printed JSON.
Context dependency versions
extensions/context.py
Bump versions: uv 0.8.22, boto3 1.40.41, fastapi 0.118.0, uvicorn 0.37.0, nuxt/ui ^4.0.0, vue-tsc ^3.1.0, eslint ^9.36.0, @types/node ^24.6.0.
Ruff config updates (repo)
ruff-test.toml, ruff.toml
Add top-level unfixable = ["SIM117"] (test config); minor formatting/comment tweaks (no rule changes) in main config.
Ruff config updates (template)
template/ruff-test.toml, template/ruff.toml
Add unfixable = ["SIM117"] (test template); reformat/comments only in main template config.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Poem

I nudge the knobs from old to new,
With jq’s glow, the logs shine through.
Ruff won’t fix what SIM117 spun,
While uv sprints to twenty-two.
A bunny taps “commit” and grins—
Small hops forward, changes in! 🐇✨

✨ Finishing touches
  • 📝 Generate Docstrings
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch fix-get-values

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between ad7c56b and 41817b7.

📒 Files selected for processing (10)
  • .copier-answers.yml (1 hunks)
  • .devcontainer/devcontainer.json (1 hunks)
  • .devcontainer/install-ci-tooling.py (1 hunks)
  • .github/workflows/get-values.yaml (1 hunks)
  • extensions/context.py (2 hunks)
  • ruff-test.toml (1 hunks)
  • ruff.toml (2 hunks)
  • template/.github/workflows/get-values.yaml (1 hunks)
  • template/ruff-test.toml (1 hunks)
  • template/ruff.toml (2 hunks)

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the project template by pulling in upstream changes from the copier template. The changes primarily focus on updating package versions and improving the GitHub Actions workflow for better debugging.

  • Updated multiple package versions including uv, boto3, fastapi, uvicorn, nuxt-ui, and others
  • Enhanced the GitHub Actions get-values workflow to use proper JSON formatting
  • Added new ruff configuration for test files to prevent auto-fixing of pytest.raises context managers

Reviewed Changes

Copilot reviewed 10 out of 10 changed files in this pull request and generated no comments.

Show a summary per file
File Description
template/ruff.toml Updated comment wording for builtins-ignorelist
template/ruff-test.toml Added unfixable configuration for SIM117 rule
template/.github/workflows/get-values.yaml Improved JSON output formatting with jq
ruff.toml Updated comment wording for builtins-ignorelist
ruff-test.toml Added unfixable configuration for SIM117 rule
extensions/context.py Updated multiple package versions
.github/workflows/get-values.yaml Improved JSON output formatting with jq
.devcontainer/install-ci-tooling.py Updated uv version
.devcontainer/devcontainer.json Updated devcontainer context hash
.copier-answers.yml Updated commit reference to v0.0.74

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@ejfine ejfine merged commit 5882376 into main Oct 1, 2025
2 of 3 checks passed
@ejfine ejfine deleted the fix-get-values branch October 1, 2025 11:04
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.

2 participants