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

Update GHA wiki page with new section "How to review GHA Pull Requests" #4867

Closed
6 tasks done
roslynwythe opened this issue Jun 21, 2023 · 17 comments
Closed
6 tasks done
Assignees
Labels
Complexity: Medium Feature: Wiki role: back end/devOps Tasks for back-end developers size: 1pt Can be done in 4-6 hours time sensitive Needs to be worked on by a particular timeframe

Comments

@roslynwythe
Copy link
Member

roslynwythe commented Jun 21, 2023

Overview

We need to add a section to the HfLA GitHub Actions wiki page to guide developers in the process of reviewing GitHub Action issues.

Action Items

  • We are currently in the process of moving the old wiki to the new website-wiki repo, so we will not be making any changes or additions to the old wiki at this time. Thus, we will be adding wiki content through a different process now. Read How to Contribute to the Wiki
  • Using the process outlined above, draft a new section of the Hack for LA's GitHub Actions page, titled "How to Review GitHub Actions Pull Requests". Include these subsections:
    • Creating a test branch on your fork
    • Modifying Repository Settings and Labels
    • Creating your own Project Board (link to Tip 6)
    • Using Personal Access Tokens to test in your own Project Board (link to Tip 7)

Resources/Instructions

Hack for LA's GitHub Actions
HfLA GitHub Actions - detail

@roslynwythe roslynwythe added role: back end/devOps Tasks for back-end developers Complexity: Medium Feature: Wiki size: 1pt Can be done in 4-6 hours role: dev leads Tasks for technical leads Draft Issue is still in the process of being created labels Jun 21, 2023
@roslynwythe roslynwythe changed the title Create New Wiki Page "How to review/test GitHub Actions" Update GHA wiki page new section "How to review GHA Pull Requests Jun 21, 2023
@roslynwythe roslynwythe changed the title Update GHA wiki page new section "How to review GHA Pull Requests Update GHA wiki page with new section "How to review GHA Pull Requests Jun 21, 2023
@roslynwythe roslynwythe changed the title Update GHA wiki page with new section "How to review GHA Pull Requests Update GHA wiki page with new section "How to review GHA Pull Requests" Jun 21, 2023
@roslynwythe roslynwythe added role: back end/devOps Tasks for back-end developers ready for product Ready for Prioritization time sensitive Needs to be worked on by a particular timeframe and removed Draft Issue is still in the process of being created role: back end/devOps Tasks for back-end developers role: dev leads Tasks for technical leads ready for product labels Jun 21, 2023
@nnr-nnr nnr-nnr self-assigned this Jul 26, 2023
@github-actions
Copy link

Hi @nnr-nnr, thank you for taking up this issue! Hfla appreciates you :)

Do let fellow developers know about your:-
i. Availability: (When are you available to work on the issue/answer questions other programmers might have about your issue?)
ii. ETA: (When do you expect this issue to be completed?)

You're awesome!

P.S. - You may not take up another issue until this issue gets merged (or closed). Thanks again :)

@nnr-nnr
Copy link
Member

nnr-nnr commented Jul 27, 2023

i. Availability: Sunday 9am–12pm PT, Monday 3pm–6pm PT
ii. ETA: Tues 8/1

@github-actions github-actions bot added the To Update ! No update has been provided label Aug 4, 2023
@github-actions
Copy link

github-actions bot commented Aug 4, 2023

@nnr-nnr

Please add update using the below template (even if you have a pull request). Afterwards, remove the 'To Update !' label and add the 'Status: Updated' label.

  1. Progress: "What is the current status of your project? What have you completed and what is left to do?"
  2. Blockers: "Difficulties or errors encountered."
  3. Availability: "How much time will you have this week to work on this issue?"
  4. ETA: "When do you expect this issue to be completed?"
  5. Pictures (optional): "Add any pictures of the visual changes made to the site so far."

If you need help, be sure to either: 1) place your issue in the Questions/In Review column of the Project Board and ask for help at your next meeting, 2) put a "Status: Help Wanted" label on your issue and pull request, or 3) put up a request for assistance on the #hfla-site channel. Please note that including your questions in the issue comments- along with screenshots, if applicable- will help us to help you. Here and here are examples of well-formed questions.

You are receiving this comment because your last comment was before Tuesday, August 1, 2023 at 12:05 AM PST.

@nnr-nnr
Copy link
Member

nnr-nnr commented Aug 4, 2023

  1. Progress: have familiarized self with pre-reads
  2. Blockers: N/A
  3. Availability: Sunday 8/6 11-1PM ET
  4. ETA: Sunday 8/6
  5. Pictures: N/A

@nnr-nnr nnr-nnr added Status: Updated No blockers and update is ready for review and removed To Update ! No update has been provided labels Aug 4, 2023
@github-actions github-actions bot removed the Status: Updated No blockers and update is ready for review label Aug 11, 2023
@github-actions github-actions bot added the To Update ! No update has been provided label Aug 18, 2023
@github-actions

This comment was marked as outdated.

@github-actions github-actions bot added 2 weeks inactive and removed To Update ! No update has been provided labels Aug 25, 2023
@github-actions

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@github-actions

This comment was marked as outdated.

@nnr-nnr nnr-nnr removed their assignment Sep 20, 2023
@pearlescence-m pearlescence-m self-assigned this Sep 20, 2023
@github-actions
Copy link

Hi @pearlescence-m, thank you for taking up this issue! Hfla appreciates you :)

Do let fellow developers know about your:-
i. Availability: (When are you available to work on the issue/answer questions other programmers might have about your issue?)
ii. ETA: (When do you expect this issue to be completed?)

You're awesome!

P.S. - You may not take up another issue until this issue gets merged (or closed). Thanks again :)

@pearlescence-m
Copy link
Member

i. Availability: In the evenings & weekends
ii. ETA: 09/23/2023 at the latest

@pearlescence-m
Copy link
Member

pearlescence-m commented Sep 23, 2023

How to Review GitHub Actions Pull Requests

Step 1: Creating a test branch on your fork

1.1. Create a new branch for testing

First, make sure your gh-pages branch in your fork is synced to the original HfLA repository.

Then, create a new branch in your updated local repository where you will apply the changes from the pull request. You can name the branch something descriptive that includes the original branch name, for example:

git checkout -b test-gha-{INSERT_NAME_OF_BRANCH} gh-pages

1.2. Pull the suggested changes:

Get the changes from the pull request into your test branch. Replace {INSERT_NAME_OF_BRANCH} with the author's branch name you want to test:

git pull https://github.com/{INSERT_NAME_OF_OWNER}/website.git {INSERT_NAME_OF_BRANCH}

1.3 Push the test branch:

Once you apply any other the issue-specific edits in .yml files necessary to trigger the GitHub Actions, push the test branch with the applied changes to your forked repository on GitHub:

git push origin test-gha-{INSERT_NAME_OF_BRANCH}

Step 2: Modifying Repository Settings and Labels

In the "Settings" of your forked repository, make sure to change your "Default branch", and select the button with the arrows to switch you default branch to the newly created branch test-gha-{INSERT_NAME_OF_BRANCH} and update to it.

Important: when you are done testing make sure to repeat these steps to re-select "gh-pages" as your default branch.

Settings page of the repository.

Settings page

There are also few other settings and configurations you may need to adjust depending on the action in review:

  • Security policies
  • Branch protection rules
  • Issues
  • Pull request (PR) templates
  • Creating necessary labels

Modifying these settings will make sure that your forked repository is aligned with the hackforla/website repository and you can properly test the changes made by the GitHub Actions. You may need to consult with the HfLA-GitHub-Actions page, because understanding action can help decide what modifications will be needed to test it.

Step 3: Creating your own Project Board

Follow the steps to create your own Project Board copy for testing.

Step 4: Using Personal Access Tokens to test in your own Project Board

You will need to create your own Personal Access Tokens in your forked repository to ensure that the workflow can access it. It is also a best practice to give your secret an expiration date.

Important: when you are creating your own token, do not give your token the same name that you see in the yaml file, otherwise unexpected behavior may result. Any other name is OK (for example, TEST_TOKEN).

Monitor the workflow runs in the "Actions" tab of your forked repository to ensure that it works as expected.

Actions tab

@pearlescence-m pearlescence-m added the Status: Help Wanted Internal assistance is required to make progress label Sep 29, 2023
@LRenDO
Copy link
Member

LRenDO commented Oct 4, 2023

Hi @pearlescence-m ! Great job with this issue! I'm assuming it's ready for review since you have completed all the steps. Adding the ready for dev lead label and adding it to the Questions / In Reveiew column as you did will notify us it's ready to be reviewed. If I got this wrong though, please let me know if there is something specific you would like help with.

  • Formatting and wording aligns well with current Wiki Docs
  • 4 topics on the issue are covered
  • Descriptions are very clear
  • Comment is linked in the How to Contribute to the Wiki page
  • The issue was moved to the correct column for review

I have a couple relatively minor requests.

  • Update the images. They are not working properly.
  • This may be a little nit-picky but, in Step 2 change you might consider adjusting depending on the action in review: to something like you may need to adjust depending on the action in review: to make it sound less optional. Most actions require creating some kind of changes to one or more of these areas to effectively test them.
  • Also in Step 2 adding something about how understanding the action can help decide what modifications will be needed to test a particular action and adding a link to the description of our GHAs

When you've made the changes you can ping me here in a comment and add the ready for dev lead label. Also, please let me know if you have any questions!

Thanks for working on this issue!

@LRenDO LRenDO removed the Status: Help Wanted Internal assistance is required to make progress label Oct 4, 2023
@pearlescence-m
Copy link
Member

@LRenDO Yes, I did finish the issue at the time, but wasn't sure if I needed to use any label at all to indicate that since it wasn't mentioned in the issue -- thanks for clarifying this and for your feedback!

I made the changes concerning points 2 and 3. However, I am not sure how to add the images here. I added direct links to the images but they are still not displaying properly. Do you know if I need to upload them to the repo's assets folder (to use in relative paths)?

@pearlescence-m pearlescence-m added the ready for dev lead Issues that tech leads or merge team members need to follow up on label Oct 5, 2023
@LRenDO
Copy link
Member

LRenDO commented Oct 5, 2023

Hi @pearlescence-m! Thanks for making the updates! They look great!

It looks like the images not working because there needs to be a blank line before the images. Like this:

<summary>Settings page of the repository.</summary> 

![Image](https://github-production-user-asset-6210df.s3.amazonaws.com/20477610/270084494-9bd8ed9d-42f4-4f17-9c7e-1ce74202c003.png)

You can also replace the word image with alt text to describe the image.

Ping me when it's all set and I will go ahead and approve it!

@pearlescence-m
Copy link
Member

@LRenDO thanks a lot, that worked! The issue is now ready for the review.

@LRenDO
Copy link
Member

LRenDO commented Oct 5, 2023

Hi @pearlescence-m! This looks great. Thanks for making the updates and contributing to the wiki! Nice work!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Complexity: Medium Feature: Wiki role: back end/devOps Tasks for back-end developers size: 1pt Can be done in 4-6 hours time sensitive Needs to be worked on by a particular timeframe
Projects
Development

No branches or pull requests

5 participants