Skip to content

Conversation

rxd90
Copy link

@rxd90 rxd90 commented Oct 6, 2025

Updated GitHub Actions workflow to configure AWS credentials and added a dry run for the release process.

Updated GitHub Actions workflow to configure AWS credentials and added a dry run for the release process.
@Copilot Copilot AI review requested due to automatic review settings October 6, 2025 19:10
Copy link
Contributor

@Copilot 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 modifies the GitHub Actions workflow for releases to support AWS credential configuration and implements a dry run approach for testing the release process.

  • Added AWS OIDC authentication to enable S3 operations during the build process
  • Commented out the original release step and replaced it with a dry run configuration
  • Added necessary permissions for OpenID Connect token usage

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

Comment on lines +28 to +31
# - name: Make release
# run: make release
# env:
# GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
Copy link

Copilot AI Oct 6, 2025

Choose a reason for hiding this comment

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

The original release step is commented out, which suggests this might be temporary for testing. Consider removing the commented code if the dry run approach is permanent, or add a comment explaining when to switch back to the full release.

Suggested change
# - name: Make release
# run: make release
# env:
# GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

Copilot uses AI. Check for mistakes.

# GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

- name: Make release (dry run)
run: goreleaser release --skip=publish --skip=validate --clean
Copy link

Copilot AI Oct 6, 2025

Choose a reason for hiding this comment

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

The dry run command skips validation which could miss potential issues. Consider removing --skip=validate to catch configuration problems early, or document why validation is being skipped.

Suggested change
run: goreleaser release --skip=publish --skip=validate --clean
run: goreleaser release --skip=publish --clean

Copilot uses AI. Check for mistakes.

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