Skip to content

Conversation

@freddydk
Copy link
Contributor

@freddydk freddydk commented Nov 17, 2024

This PR is the first part of supporting GitHub Apps for authentication instead of PATs in GhTokenWorkflow.

After this PR, we still need a PAT for running the End 2 End tests, and we will have to have a PAT for End 2 End tests until we no longer support the versions, which doesn't support using GitHub Apps for authentication. The full scenario end 2 end test does use the GitHub app for authentication and does as such test that GhTokenWorkflow can be a GitHub app.

Also, the microsoft owned repositories used in end 2 end (bcsamples-*) are also using the PAT, as we currently do not have the GitHub App installed in the microsoft org. This will be part of a second part of this - to use the PAT in fewer places in the end 2 end tests.

TODO:

  • Documentation

@freddydk freddydk requested a review from mazhelez January 28, 2025 13:13
aholstrup1
aholstrup1 previously approved these changes Jan 30, 2025
mazhelez
mazhelez previously approved these changes Jan 30, 2025
Copy link
Collaborator

@mazhelez mazhelez left a comment

Choose a reason for hiding this comment

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

Added minor comments.

@freddydk freddydk dismissed stale reviews from mazhelez and aholstrup1 via 9526def January 30, 2025 09:22
mazhelez
mazhelez previously approved these changes Jan 30, 2025
aholstrup1
aholstrup1 previously approved these changes Jan 30, 2025
@freddydk freddydk dismissed stale reviews from aholstrup1 and mazhelez via 1ca0e03 January 30, 2025 19:39
@freddydk freddydk merged commit 7be5005 into microsoft:main Jan 31, 2025
5 checks passed
@freddydk freddydk deleted the noPAT branch April 25, 2025 11:41
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