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 GITHUB_TOKEN secret #2869

Merged
merged 1 commit into from Nov 30, 2022

Conversation

quoideneuf
Copy link
Collaborator

Fixes an incorrect GitHub Actions secret reference, and adds an option to fetch stats by pushing to fetches-download-stats-on-push. Also includes last month's download.

Copy link
Collaborator

@brianzelip brianzelip left a comment

Choose a reason for hiding this comment

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

In looking at the ad-m/github-push-action docs, I see they set persist-credentials: false during the checkout step, which impacts the secrets.GITHUB_TOKEN used in the Push changes step.

Were you able to run the new workflow and see it work as expected?

Copy link
Collaborator

@brianzelip brianzelip left a comment

Choose a reason for hiding this comment

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

Also, I don't have access to the repo's settings, so I can't review the permissions for the GITHUB_TOKEN as described in the docs.

@quoideneuf
Copy link
Collaborator Author

It appears to work in my repository, but I can't say I know for sure that it will work in this one. I hope so!

@brianzelip
Copy link
Collaborator

This just needs rebasing w/ master

@quoideneuf quoideneuf merged commit 3b6ebc0 into archivesspace:master Nov 30, 2022
@quoideneuf quoideneuf deleted the fixup-stats-workflow branch November 30, 2022 18: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.

None yet

2 participants