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

Fixes #2220 #3426

Closed

Conversation

poorvi4
Copy link
Member

@poorvi4 poorvi4 commented Jul 25, 2022

Fixes #2220

What changes did you make and why did you make them ?

Implemented solutions as suggested in #2220:

  • Renamed 'add-update-label-weekly.yml'' to 'schedule-fri-0700.yml'' and updated 'name' env variable to Schedule Friday 0700
  • Moved add-update-label-weekly directory to already existing /github-actions/trigger-schedule/ directory
  • Updated path to respective add-label.js to reflect current directory structure

Much of the changes are purely for organizational purposes to better maintain the workflows/actions created

Screenshots of Proposed Changes Of The Website (if any, please do not screen shot code changes)

No changes since the directories were reorganized and minor changes were made to accompanying files to better maintain code. Function remains same.

NOTE: One thing to mention is that I'm not entirely sure how to locally test a github action/workflow. If anyone knows, please let me know! I've found this tool neko which but I haven't used/tested it myself yet.

@github-actions
Copy link

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.

git checkout -b poorvi4-add-update-label-restructure-2220 gh-pages
git pull https://github.com/poorvi4/website.git add-update-label-restructure-2220

@github-actions github-actions bot added Feature: Refactor GHA Refactoring GitHub actions to fit latest architectural norms role: back end/devOps Tasks for back-end developers size: 1pt Can be done in 4-6 hours Complexity: Medium Status: Updated No blockers and update is ready for review labels Jul 25, 2022
@arpitapandya arpitapandya self-requested a review July 25, 2022 16:33
@arpitapandya
Copy link
Member

Availability: 6 Hours
ETA: EOD 2022-07-26

@BeckettOBrien BeckettOBrien self-requested a review July 25, 2022 17:29
@BeckettOBrien
Copy link
Member

Availability: 5 Hours
ETA: EOD 2022-07-25

BeckettOBrien
BeckettOBrien previously approved these changes Jul 25, 2022
Copy link
Member

@BeckettOBrien BeckettOBrien left a comment

Choose a reason for hiding this comment

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

All the changes look correct, so good job! And thanks for letting me know about that tool to test actions, it looks really helpful!

@arpitapandya
Copy link
Member

Where do we test it? Any specific tool? @BeckettOBrien @poorvi4

@blulady
Copy link
Member

blulady commented Jul 25, 2022

The Act from Nektos tool does look really helpful. Did you try it @BeckettOBrien?

@BeckettOBrien
Copy link
Member

I did, but so far I only tested it on a different action because I'm still figuring out how to trigger specific scheduled actions or if I have to temporarily add a workflow_dispatch first.

@arpitapandya
Copy link
Member

arpitapandya commented Jul 27, 2022

Hey @poorvi4 sorry, I am working on it, I am actually waiting on the GitHub action testing demo as discussed during our last team meeting as I am also having issues in testing. Once we figure that out I will review it. You may ask the team lead if you have to work on next issue since I holding this off for a while.

@Zak234 Zak234 requested review from Zak234 and removed request for Zak234 July 29, 2022 02:23
Copy link
Contributor

@Zak234 Zak234 left a comment

Choose a reason for hiding this comment

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

I checked the changes and it look like you created a new file named "schedule-fri-0700.yml" instead of renaming "add-update-label-weekly.yml" as both files still show up in workflows.

@poorvi4 poorvi4 requested a review from Zak234 July 31, 2022 21:01
@poorvi4
Copy link
Member Author

poorvi4 commented Jul 31, 2022

Oh good catch! I updated the pull request so the add-update-label file should be correctly renamed to schedule-fri-0700

Copy link
Contributor

@Zak234 Zak234 left a comment

Choose a reason for hiding this comment

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

When I run the action, I keep getting this error:

Unhandled error: Error: Cannot find module '../utils/find-linked-issue'
Require stack:
- /home/runner/work/website/website/github-actions/trigger-schedule/add-update-label-weekly/add-label.js
- /home/runner/work/_actions/actions/github-script/v4/dist/index.js

Have you experienced this error while running the action?

@Zak234
Copy link
Contributor

Zak234 commented Aug 5, 2022

I think the problem is that line 302 is
const path = './github-actions/add-update-label-weekly/update-instructions-template.md'
and it should be
const path = './github-actions/trigger-schedule/add-update-label-weekly/update-instructions-template.md'

@arpitapandya arpitapandya removed their request for review August 13, 2022 21:53
@blulady
Copy link
Member

blulady commented Aug 24, 2022

Hey @poorvi4, just wondering if you are still going to be able to finish this pull request. Please post an update or let us know if you won't be able to continue working on the PR.

@poorvi4
Copy link
Member Author

poorvi4 commented Aug 26, 2022

Hi sorry for the delay I’ll finish it up in the coming days

@poorvi4 poorvi4 requested a review from Zak234 August 30, 2022 03:26
@poorvi4
Copy link
Member Author

poorvi4 commented Aug 30, 2022

@Zak234 @blulady Hi all, I updated the path as per Sakari's suggestion so everything should be good to go!

@blulady blulady self-requested a review September 9, 2022 02:23
@blulady blulady removed the request for review from Zak234 October 10, 2022 17:32
Copy link
Member

@blulady blulady left a comment

Choose a reason for hiding this comment

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

Hey @poorvi4,
Sorry it has taken me so long to get back to you, I've been wrestling with getting the project board locally. So I am currently running this test in my repo and I keep getting the error
`
Add-Update-Label-to-Issues-Weekly
Unhandled error: Error: Cannot find module '../utils/find-linked-issue'
Require stack:

  • /home/runner/work/website/website/github-actions/trigger-schedule/add-update-label-weekly/add-label.js
  • /home/runner/work/_actions/actions/github-script/v4/dist/index.js`
    Which is odd because utils is in the same place it was relative to where the new file is.

Did you want to keep working on this or should we hand it off to someone else?

@blulady

This comment was marked as outdated.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Complexity: Medium Feature: Refactor GHA Refactoring GitHub actions to fit latest architectural norms role: back end/devOps Tasks for back-end developers size: 1pt Can be done in 4-6 hours Status: Updated No blockers and update is ready for review
Projects
Development

Successfully merging this pull request may close these issues.

GitHub Actions: Rename "Add Update Label to Issues Weekly" to "Schedule Friday 0700"
5 participants