-
Notifications
You must be signed in to change notification settings - Fork 491
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 PR artifacts workflow #2028
Fix PR artifacts workflow #2028
Conversation
e7a7661
to
352553c
Compare
352553c
to
7563d5d
Compare
I remember hearing the artifact builds would get removed after 5 days. Is this the same for nightly? |
7563d5d
to
8585d84
Compare
The artifacts actually by default expire after 90 days, but I think there is a github UI bug or something that shows that they are all expired when viewing from a specific page |
Ah ok, I think 90 days is more than sufficient for now. |
8585d84
to
23573ac
Compare
23573ac
to
f649fc1
Compare
Working on local fork here: garrettjoecox#26 Won't work here until it's merged into develop (github actions requirement) |
Found the solution to determining the PR number here oprypin/nightly.link#37, specifically this old commit: https://github.com/oprypin/nightly.link/blob/4e83d1452dd1bf350ebe1a3910e395e6cf447fdf/.github/workflows/pr-comment.yml
This also updates the workflow to post nightly.link links, as the current links still require you to be authenticated with github