Skip to content

Add github-handle variable to Jessie Liu#7299

Merged
t-will-gillis merged 1 commit intohackforla:gh-pagesfrom
minkang3:github-handle-for-jessie-liu-7250
Aug 18, 2024
Merged

Add github-handle variable to Jessie Liu#7299
t-will-gillis merged 1 commit intohackforla:gh-pagesfrom
minkang3:github-handle-for-jessie-liu-7250

Conversation

@minkang3
Copy link
Member

Fixes #7250

What changes did you make?

  • Added variable github-handle for Jessie Liu in _projects/tech-work-experience.md

Why did you make the changes (we will use this info to test)?

  • To reduce redundancy within the file, combining github and picture to just one variable, github-handle.

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

No visual changes.

@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 minkang3-github-handle-for-jessie-liu-7250 gh-pages
git pull https://github.com/minkang3/website.git github-handle-for-jessie-liu-7250

@github-actions github-actions bot added good first issue Good for newcomers role: front end Tasks for front end developers role: back end/devOps Tasks for back-end developers P-Feature: Project Info and Page A project's detail page (e.g. https://www.hackforla.org/projects/100-automations) size: 0.25pt Can be done in 0.5 to 1.5 hours labels Aug 15, 2024
@mrodz mrodz self-requested a review August 15, 2024 21:29
Copy link
Member

@mrodz mrodz left a comment

Choose a reason for hiding this comment

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

Hello @minkang3, thank you for picking this up!

  • Correctly inserted github-handle YAML property to Jessie Liu's object, used spaces instead of tabs
  • Correct to/from branches
  • Linked issue number
  • Good explanation of changes

Approving because this PR looks solid. Good luck with future tickets!

@mmcclanahan mmcclanahan self-assigned this Aug 18, 2024
@mmcclanahan
Copy link
Member

Review ETA: 10pm 8/17/24
Availability: Sunday all day

Copy link
Member

@mmcclanahan mmcclanahan left a comment

Choose a reason for hiding this comment

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

@minkang3, good work!

To and from branches are correct
Used spaces to indent the github-handle
Correctly linked issue
Clear why and how explanation of changes
No visual changes to website

Nice!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

good first issue Good for newcomers P-Feature: Project Info and Page A project's detail page (e.g. https://www.hackforla.org/projects/100-automations) role: back end/devOps Tasks for back-end developers role: front end Tasks for front end developers size: 0.25pt Can be done in 0.5 to 1.5 hours

Projects

Development

Successfully merging this pull request may close these issues.

Add github-handle for Jessie Liu in tech-work-experience.md

4 participants