-
-
Notifications
You must be signed in to change notification settings - Fork 778
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
Update schedule-daily-1100.yml #5513
Update schedule-daily-1100.yml #5513
Conversation
Post test, update OK
Want to review this pull request? Take a look at this documentation for a step by step guide! From your project repository, check out a new branch and test the changes.
|
ETA: By next Monday |
ETA: By next Monday |
unable to do this PR, removed name off assignee list. |
Sorry I am unable to complete this PR. I have removed my name from reviewers list |
ETA: Tuesday 10/10, EOD |
I still plan to review this - my family has been sick over the long weekend so I'm behind. My new, very cautious, ETA is Friday 10/13 EOD. I am typically available via Slack during the day, 9-5pm. |
ETA: By Monday 10/16 |
Availability: Sunday - Saturday 2pm - 7pm PST |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@t-will-gillis This PR looks good - the only code changed was actions/checkout@v3 to actions/checkout@v4 in schedule-daily-1100.yml, as requested in the issue. I tested the change locally and the schedule-daily-1100 workflow ran successfully with the change to actions/checkout@v4. Nice work!
Link to successful run WITH the v4 change: https://github.com/allison-truhlar/website/actions/runs/6512541496/job/17690398322
Link to successful run prior to the v4 change (for comparison): https://github.com/allison-truhlar/website/actions/runs/6512518673/job/17690321198
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry - I hit comment instead of approve! Copying my comment below for reference
@t-will-gillis This PR looks good - the only code changed was actions/checkout@v3 to actions/checkout@v4 in schedule-daily-1100.yml, as requested in the issue. I tested the change locally and the schedule-daily-1100 workflow ran successfully with the change to actions/checkout@v4. Nice work!
Link to successful run WITH the v4 change: https://github.com/allison-truhlar/website/actions/runs/6512541496/job/17690398322
Link to successful run prior to the v4 change (for comparison): https://github.com/allison-truhlar/website/actions/runs/6512518673/job/17690321198
Hey @brianf4 @k-rewd @dineshsk98 just checking in to see whether you have had a chance to get started on your reviews yet, and if so your timeframes for finishing? If you have already started reviewing this, please let me know if you have questions or how I can help you. I would like to wrap this PR up in the next day or two because there are currently 3 additional PRs that are waiting on this one to be completed. If you have not started reviewing this yet, I would like to ask if you would consider un-assigning yourself from this one. There will be many more PRs involving GHAs coming up. (This includes #5637 which is over two weeks old and needs a review. ) |
Sorry, for the delay. I started the review, but got stuck with other work. I will unassign myself off of this one. |
@t-will-gillis Sorry got caught up with work as well but currently working on it. Will try to have it reviewed by tonight or by tomorrow. |
Hey @dineshsk98 and @brianf4 - thanks for the notes and updates |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @t-will-gillis, I just ran the workflow changes locally for the schedule-daily-1100.yml file and everything checks out. The workflow ran successfully with no errors on my end.
Fixes #5494
What changes did you make?
actions/checkout@v3
-->actions/checkout@v4
inschedule-daily-1100.yml
Why did you make the changes (we will use this info to test)?
Supporting file
Note to PR Reviewers
CONTRIBUTING.md
document. You will want to have an environment set up in your local repo and test the GHAs on the desktop..github/workflows/schedule-daily-1100.yml
:workflow_dispatch:
inline with Line 5'sschedule:
if: github.repository == 'your-repo-name/website'
uses: actions/checkout@v4
token: ${{ secrets.TEST_TOKEN }}
_data/external/github-data.json