Skip to content

Revert "Update winget.yml to use app access token"#805

Merged
devm33 merged 1 commit intomainfrom
revert-764-devm33/winget-app
Dec 17, 2025
Merged

Revert "Update winget.yml to use app access token"#805
devm33 merged 1 commit intomainfrom
revert-764-devm33/winget-app

Conversation

@devm33
Copy link
Copy Markdown
Member

@devm33 devm33 commented Dec 17, 2025

Reverts #764

Copilot AI review requested due to automatic review settings December 17, 2025 21:13
Copy link
Copy Markdown
Contributor

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 reverts changes from PR #764, rolling back the GitHub workflow from using a dynamically generated app access token to using a pre-configured secret token for WinGet package submissions.

Key Changes:

  • Removed the actions/create-github-app-token@v2 step that generated tokens dynamically from app credentials
  • Restored the job-level environment variable configuration using secrets.WINGET_CREATE_GITHUB_TOKEN

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


# winget-create is only supported on Windows
runs-on: windows-latest

Copy link

Copilot AI Dec 17, 2025

Choose a reason for hiding this comment

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

This line contains trailing whitespace. Consider removing it to maintain clean code formatting.

Suggested change

Copilot uses AI. Check for mistakes.
Copy link
Copy Markdown
Contributor

@andyfeller andyfeller left a comment

Choose a reason for hiding this comment

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

Sadly this is a limitation of apps 🤦

@devm33 devm33 merged commit 6118464 into main Dec 17, 2025
9 checks passed
@devm33 devm33 deleted the revert-764-devm33/winget-app branch December 17, 2025 21:35
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