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 actions/checkout@v3 token for git push to trigger other actions #180

Merged
merged 6 commits into from
May 2, 2023

Conversation

houstonj1
Copy link
Contributor

@houstonj1 houstonj1 commented May 2, 2023

@devth I think this will configure git correctly on checkout, so we can push 🤞

@devth devth merged commit 9b46f12 into devth:master May 2, 2023
@devth
Copy link
Owner

devth commented May 2, 2023

@houstonj1
Copy link
Contributor Author

@devth I found this - actions/checkout#664 (comment)

Mind checking if that PAT has an expire date set? Other than that, we can try adding the specific repo to checkout and the branch to the checkout action config

@devth
Copy link
Owner

devth commented May 2, 2023

Nope, it doesn't have an expiration date!

@houstonj1
Copy link
Contributor Author

Kinda lame it needs one but, I guess that is good for security

@devth
Copy link
Owner

devth commented May 2, 2023

Let me provision a new one with exp date.

@devth
Copy link
Owner

devth commented May 2, 2023

Ok, new one is set. Expires in a year. It has read/write for commits on this repo.

@houstonj1
Copy link
Contributor Author

🎉 It worked, but we didn't get the tag with it.... Something to look at next

@devth
Copy link
Owner

devth commented May 2, 2023

Amazing! Thanks for fixing it.

Hope it can auto close the github issue too at some point. I think that was working at some point? We'll see with the next release I guess!

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