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

fix: Pass the github token to all jobs in the workflows. #1166

Merged
merged 1 commit into from
Jun 20, 2024

Conversation

feanil
Copy link
Contributor

@feanil feanil commented Jun 18, 2024

We weren't passing the GITHUB_TOKEN to both steps of the quarterly task
which was causing it to fail on the first step which didn't pass the
GITHUB_TOKEN.

Rather than just fixing the one step, I updated both the quarterly and
weekly file to pass the GITHUB_TOKEN env to all jobs/steps in their
respective workflow files. Since these files should be only github
issue creation commands I think this is fine to do and simplifes things
and makes the workflows less error prone.

We weren't passing the GITHUB_TOKEN to both steps of the quarterly task
which was causing it to fail on the first step which didn't pass the
GITHUB_TOKEN.

Rather than just fixing the one step, I updated both the quarterly and
weekly file to pass the GITHUB_TOKEN env to all jobs/steps in their
respective workflow files.  Since these files should be only github
issue creation commands I think this is fine to do and simplifes things
and makes the workflows less error prone.
Copy link
Contributor

@arbrandes arbrandes left a comment

Choose a reason for hiding this comment

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

LGTM!

@feanil feanil merged commit 7ac6640 into main Jun 20, 2024
2 checks passed
@feanil feanil deleted the feanil/fix_workflows branch June 20, 2024 12:01
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