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

Use Github App credentials for issue commands workflow #280

Merged
merged 1 commit into from
Sep 26, 2023
Merged

Conversation

katebrenner
Copy link
Contributor

Properly fix workflow - same as grafana/grafana-iot-twinmaker-app#222

@@ -17,8 +17,14 @@ jobs:
ref: main
- name: Install Actions
run: npm install --production --prefix ./actions
- name: "Generate token"
id: generate_token
uses: tibdex/github-app-token@b62528385c34dbc9f38e5f4225ac829252d1ea92
Copy link
Member

@sarahzinger sarahzinger Sep 26, 2023

Choose a reason for hiding this comment

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

what's this? and why not something like tibdex/github-app-token@v2 is this a specific git commit that we need for some reason?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This action turns the private key from the app into a token that's usable for running the issue_commands. But I actually don't know why the recommendation was to use that particular commit. This is a good callout! I will ask

@katebrenner katebrenner merged commit e399ce2 into main Sep 26, 2023
2 checks passed
@katebrenner katebrenner deleted the app_token branch September 26, 2023 17:15
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.

2 participants