Consolidate and update dependency and CI packages#5260
Open
nllong wants to merge 2 commits into
Open
Conversation
Contributor
There was a problem hiding this comment.
Pull request overview
This PR consolidates routine dependency and CI maintenance by updating pinned Python packages (and regenerating uv.lock), refreshing GitHub Actions versions used in CI/publish workflows, and adjusting the ECS Docker build’s Node install behavior.
Changes:
- Bump multiple Python dependencies and dev tools (e.g., boto3/botocore, pytest, coverage, sentry-sdk, django-debug-toolbar) and update
uv.lockaccordingly. - Update GitHub Actions used in CI/publish workflows (checkout and cache major versions).
- Change the ECS Docker image build to run
pnpm installwith--ignore-scriptsduring the Angular build phase.
Reviewed changes
Copilot reviewed 4 out of 5 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
uv.lock |
Updates the locked Python dependency set to the newly bumped versions. |
pyproject.toml |
Updates pinned Python dependency versions to match the lockfile refresh. |
Dockerfile.ecs |
Changes pnpm install flags used before building the Angular submodule during image build. |
.github/workflows/publish.yml |
Updates actions/checkout and actions/cache versions for publish workflow steps. |
.github/workflows/ci.yml |
Updates actions/checkout and actions/cache versions for CI jobs and includes a minor formatting-only change. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Comment on lines
+78
to
80
| RUN pnpm install --frozen-lockfile --ignore-scripts && \ | ||
| pnpm -C /seed/ng_seed/seed-angular build && \ | ||
| pnpm install --prod --frozen-lockfile --ignore-scripts && \ |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Any background context you want to provide?
Combine multiple PRs with deps and CI updates.