Skip to content

fix: use PAT for release workflow#113

Merged
saralvasquez merged 1 commit into
mainfrom
release-workflow
Jun 8, 2026
Merged

fix: use PAT for release workflow#113
saralvasquez merged 1 commit into
mainfrom
release-workflow

Conversation

@saralvasquez

Copy link
Copy Markdown
Contributor

Summary of changes

This changes the release workflow to use a personal access token for the git commands at the end of the release workflow. This should resolve the issue where the version change push to main fails due to the ruleset requirements we have in place. The PAT is coming from a secure new github account/user that only the SDK team has access to. This user has been included in the bypass list for the ruleset and I've added the token to the repo secrets

AI Usage

Which AI Agent Was Used?

  • Copilot
  • Claude
  • Other (Type Name Here)

How was AI used?
no AI was used

Estimated AI Code Contribution

  • less than 30%
  • 30 - 60%
  • 60 - 100%

Checklist

  • Added a changelog entry
  • Tested and confirmed payment flows affected by this change are functioning as expected

Authors

@saralvasquez saralvasquez requested a review from a team June 5, 2026 18:57
@saralvasquez saralvasquez requested a review from a team as a code owner June 5, 2026 18:57
@github-actions

github-actions Bot commented Jun 5, 2026

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities found.

Scanned Files

None

@noguier noguier left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🎊

@noguier noguier left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Didn't notice the failing test when approving, would you mind taking a look?

@saralvasquez

Copy link
Copy Markdown
Contributor Author

It's the same flaky tests that I was dealing with on the browser switch PR. The fix is on that branch if you wanna take a look. The page just renders too slow so the assertion fails

@noguier

noguier commented Jun 5, 2026

Copy link
Copy Markdown
Contributor

It's the same flaky tests that I was dealing with on the browser switch PR. The fix is on that branch if you wanna take a look.

thank you , that’s what I thought but wanted to double check

@noguier noguier left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

TY 🙏🏼

@saralvasquez saralvasquez merged commit 6463d40 into main Jun 8, 2026
11 of 14 checks passed
@saralvasquez saralvasquez deleted the release-workflow branch June 8, 2026 17:25
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.

3 participants