Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Better work this time... #1607

Merged
merged 2 commits into from
Jan 27, 2025
Merged

Better work this time... #1607

merged 2 commits into from
Jan 27, 2025

Conversation

kalverra
Copy link
Collaborator

@kalverra kalverra commented Jan 27, 2025


Below is a summarization created by an LLM (gpt-4-0125-preview). Be mindful of hallucinations and verify accuracy.

Why

The changes aim to enhance the release workflow and documentation for the project. By specifying an environment for the release job and updating permissions, the workflow's integration with AWS services is improved. Additionally, updating the IMAGE_PREFIX to use a different AWS account number for QA purposes reflects a more refined CI/CD process. The introduction of a new changeset file and the correction of a filename typo in another changeset document further streamline release documentation and version tracking.

What

  • .github/workflows/parrot-release.yml
    • Added environment: integration to specify the environment for the release job.
    • Updated permissions with contents: write to enhance interaction with the repository content.
    • Changed IMAGE_PREFIX from using ${{ secrets.AWS_ACCOUNT_NUMBER }} to ${{ secrets.QA_AWS_ACCOUNT_NUMBER }} for the Docker image, indicating a shift towards a QA-specific AWS account.
  • parrot/.changeset/v0.1.7.md
    • Created a new changeset file with content "- More release pipeline fixes!" to document updates in the release pipeline.
  • Deleted parrot/.changeset/v01.7.md
    • Removed a previously existing changeset file due to a filename typo or duplication.

@cl-sonarqube-production
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@kalverra kalverra merged commit fee259e into main Jan 27, 2025
56 of 58 checks passed
@kalverra kalverra deleted the goreleaserPermissions branch January 27, 2025 19:02
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