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 nais-team-app instead of srvnais PAT #9

Merged
merged 2 commits into from
Oct 15, 2021
Merged

use nais-team-app instead of srvnais PAT #9

merged 2 commits into from
Oct 15, 2021

Conversation

chinatsu
Copy link
Contributor

more info: https://github.com/apps/nais-team-app

i should have added the necessary secrets (NAIS_APP_PRIVATE_KEY, and NAIS_APP_ID), but it's probably smart to have a look at the repository secrets and verify their existence before merging this.

also, i removed PUSH_TOKEN from the goreleaser step. i couldn't find the env variable documented anywhere in the goreleaser project, and i want to eliminate the use of SRVNAIS_REPO_PUSH_PAT

more info: https://github.com/apps/nais-team-app

i should have added the necessary secrets (`NAIS_APP_PRIVATE_KEY`, and `NAIS_APP_ID`), but it's probably smart to have a look at the [repository secrets](../settings/secrets/actions) and verify their existence before merging this.

also, i removed PUSH_TOKEN from the goreleaser step. i couldn't find the env variable documented anywhere in the goreleaser project, and i want to eliminate the use of `SRVNAIS_REPO_PUSH_PAT`
@mortenlj
Copy link
Contributor

PUSH_TOKEN is used by goreleaser here:

token: "{{ .Env.PUSH_TOKEN }}"

It is used to push to https://github.com/nais/homebrew-tap.

Copy link
Contributor

@mortenlj mortenlj left a comment

Choose a reason for hiding this comment

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

goreleaser needs a token in order to push to nais/homebrew-tap.

@chinatsu chinatsu requested a review from mortenlj October 14, 2021 11:09
@chinatsu chinatsu merged commit fac1e28 into master Oct 15, 2021
@chinatsu chinatsu deleted the app-token branch October 15, 2021 06:00
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