Skip to content

Conversation

@nolan1999
Copy link
Member

No description provided.

@nolan1999 nolan1999 self-assigned this Oct 4, 2025
@nolan1999 nolan1999 requested a review from Copilot October 4, 2025 14:02
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 refactors the deployment pipeline by splitting a monolithic workflow into separate GitHub and ECR publishing workflows, and changes the AWS account ID from g0e9g3b1 to w2b7b8c0 across all container image references.

  • Split the original publish-version.yaml workflow into separate GitHub and ECR publishing workflows
  • Updated AWS account ID in all ECR image URLs from g0e9g3b1 to w2b7b8c0
  • Enhanced ECR workflow with improved version detection and conditional latest tag management

Reviewed Changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
application_config.yaml Updated ECR registry URLs to use new AWS account ID
README.md Updated ECR registry URL in documentation example
.github/workflows/publish-version.yaml Removed original monolithic workflow
.github/workflows/publish-version-gh.yaml Added new GitHub-only publishing workflow
.github/workflows/publish-version-ecr.yaml Added new ECR-only publishing workflow with enhanced logic

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

@github-actions
Copy link

github-actions bot commented Oct 4, 2025

Coverage

Coverage Report
FileStmtsMissCoverMissing
api
   __init__.py00100% 
   database.py13469%22–26
   dependencies.py512159%32–37, 59, 69, 78–97, 103–115
   exceptions.py9278%10, 18
   main.py24388%35, 45, 50
   models.py430100% 
   settings.py772370%15–18, 28–29, 65–69, 74, 78, 86, 95–101, 104–109, 112, 121
   tags.py10100% 
api/core
   __init__.py00100% 
   aws.py5180%7
   filesystem.py2253385%35, 48, 51, 54, 59, 66, 72, 87, 90, 93, 96, 99, 109, 114, 194, 196–205, 219, 291, 348, 356–368, 379, 410
   notifications.py271352%14, 28–29, 32–52
api/crud
   __init__.py00100% 
   job.py71494%30, 56, 144–145
api/endpoints
   __init__.py00100% 
   auth.py654925%30–69, 79–106, 118–155
   auth_get.py14286%22, 39
   files.py52688%26–27, 42–43, 112, 128
   job_update.py29293%28, 45
   jobs.py34294%24, 88
api/schemas
   __init__.py00100% 
   file.py240100% 
   job.py116992%27, 33, 36, 42, 47, 76, 85, 91, 93
   job_update.py60100% 
   token.py40100% 
   user.py140100% 
TOTAL90417481% 

Tests Skipped Failures Errors Time
128 0 💤 0 ❌ 0 🔥 12m 10s ⏱️

@nolan1999 nolan1999 requested a review from Copilot October 4, 2025 15:38
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

Copilot reviewed 8 out of 8 changed files in this pull request and generated 2 comments.


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

nolan1999 and others added 2 commits October 4, 2025 17:40
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@nolan1999 nolan1999 merged commit 9ddbb55 into main Oct 4, 2025
2 checks passed
@nolan1999 nolan1999 deleted the refactor_deployment branch October 4, 2025 16:01
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