Skip to content

Update CI workflow for package type and S3 upload URI#155

Merged
nbolton merged 3 commits intomasterfrom
dev-builds
Feb 17, 2026
Merged

Update CI workflow for package type and S3 upload URI#155
nbolton merged 3 commits intomasterfrom
dev-builds

Conversation

@nbolton
Copy link
Copy Markdown
Member

@nbolton nbolton commented Feb 17, 2026

Enhance the CI workflow by introducing a new input for package type, allowing options for snapshot, release, or dev. Update the S3 upload process to use a unified URI format for better organization of uploaded files.

Copy link
Copy Markdown

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 modernizes the CI workflow inputs and S3 upload configuration. It replaces the boolean test-release input with a more flexible package-type choice input supporting three build types (snapshot, release, dev), and consolidates the separate aws-s3-bucket and aws-s3-upload-dir inputs into a single aws-s3-uri input for simpler S3 path configuration.

Changes:

  • Replaced test-release boolean input with package-type choice input offering snapshot, release, and dev options
  • Consolidated aws-s3-bucket and aws-s3-upload-dir into a unified aws-s3-uri input
  • Updated version environment variable logic to use the new package-type input values
  • Enhanced S3 upload script to support dev package type and use the new unified URI format

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/ci.yml Outdated
Comment thread .github/workflows/ci.yml
Comment thread .github/workflows/ci.yml Outdated
@nbolton nbolton merged commit 804f1e9 into master Feb 17, 2026
23 checks passed
@nbolton nbolton deleted the dev-builds branch February 17, 2026 11:05
@github-actions
Copy link
Copy Markdown

✅ Merge build was successful.
Version: 1.20.1-snapshot+r4
Run: CI (success)

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