Skip to content

ci(github): update devops template workflows#24

Merged
ncipollina merged 4 commits into
mainfrom
ci/update-devops-template-workflows
Jul 22, 2026
Merged

ci(github): update devops template workflows#24
ncipollina merged 4 commits into
mainfrom
ci/update-devops-template-workflows

Conversation

@ncipollina

Copy link
Copy Markdown
Contributor

Summary

Updates the remaining Lambda hosting workflows from the old devops-templates@v6.2 references to v10.2. This also removes inputs that are not valid for the reusable workflows and keeps the package build docs path filters aligned with the uv-based docs dependency files.

Changes

  • Updated .github/workflows/pr-build.yaml to use LayeredCraft/devops-templates/.github/workflows/pr-build.yaml@v10.2.
  • Removed the unsupported testDirectory input from the PR build reusable workflow call.
  • Updated .github/workflows/build.yaml to use LayeredCraft/devops-templates/.github/workflows/package-build.yaml@v10.2.
  • Removed unsupported package-build inputs: useMtpRunner and testDirectory.
  • Replaced the old requirements.txt path ignore with pyproject.toml and uv.lock.

Validation

  • Reviewed reusable workflow inputs against the current devops-templates workflow definitions.
  • No local build required; workflow configuration only.

@chatgpt-codex-connector chatgpt-codex-connector Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: dbb731f7e1

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .github/workflows/pr-build.yaml Outdated
jobs:
build:
uses: LayeredCraft/devops-templates/.github/workflows/pr-build.yaml@v6.2
uses: LayeredCraft/devops-templates/.github/workflows/pr-build.yaml@v10.2

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P1 Badge Add MTP runner selection before switching PR template

With this upgrade, the v10.2 reusable PR workflow takes the existing useMtpRunner: true/enableCodeCoverage: true inputs and runs its MTP coverage path (dotnet test --solution ... --report-xunit-trx --coverage). The dotnet test CLI docs say MTP mode must be selected in global.json, while VSTest remains the default; I checked the repo with rg --files and there is no global.json, so PR builds will run the default VSTest parser and fail on the MTP-only options. Add the global.json test runner selection, or avoid the v10.2 MTP path until the repo opts into it.

Useful? React with 👍 / 👎.

@ncipollina
ncipollina merged commit 0897922 into main Jul 22, 2026
2 checks passed
@ncipollina
ncipollina deleted the ci/update-devops-template-workflows branch July 22, 2026 18:51
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.

1 participant