Skip to content

Add Github Handle for Nino Dzotsenidze in tech-work-experience.md#7306

Merged
jphamtv merged 1 commit intohackforla:gh-pagesfrom
izma-mujeeb:add-github-handle-7253
Aug 22, 2024
Merged

Add Github Handle for Nino Dzotsenidze in tech-work-experience.md#7306
jphamtv merged 1 commit intohackforla:gh-pagesfrom
izma-mujeeb:add-github-handle-7253

Conversation

@izma-mujeeb
Copy link
Member

@izma-mujeeb izma-mujeeb commented Aug 16, 2024

Fixes #7253

What changes did you make?

  • Using spaces, I added a variable to hold the github-handle for Nino Dzotsenidze

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

  • The reason why I need to add a single variable holding the github-handle is because eventually this variable will replace the github and picture variables. This is being done for every member of the leadership team to ensure that we are writing efficient code and reducing redundancy.

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

No visual changes to the website

@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 izma-mujeeb-add-github-handle-7253 gh-pages
git pull https://github.com/izma-mujeeb/website.git add-github-handle-7253

@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 16, 2024
@mrodz
Copy link
Member

mrodz commented Aug 18, 2024

Availability: Weekdays 5pm-9pm, Weekends 10am-5pm
ETA: EOD 8/18

@8alpreet
Copy link
Member

8alpreet commented Aug 20, 2024

Review ETA: EOD
Availability: Mon to Fri; 3 p.m. to 7 p.m.

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.

Hi @izma-mujeeb! Good work on this issue.

  • Correctly linked issue number
  • Correct to/from branches
  • Correctly modified Nino Dzotsenidze's yaml object, added github-handle
  • Correctly used spaces
  • Good description of the issue (for future issues, if you don't have enough items in the bullet point list, you can remove the extra -)

Keep it up :)

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.

Hey @izma-mujeeb!

Great job adding the github-handle.
One quick thing that could be changed for clarity is the PR name. Adding whose github handle variable was added would be helpful for readers. Such as: Added Github Handle Variable for Nino Dzotsenidze

Copy link
Member

@8alpreet 8alpreet left a comment

Choose a reason for hiding this comment

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

Hi @izma-mujeeb,

What went well:

  • Into branch is set up correctly.
  • PR is linked to the correct issue number.
  • Branch is named correctly
  • Only relevant lines of code were changed.

Changes needed:

  • In the "why did you make these changes" section
  • You're correct in stating the purpose of the "github-handle" key; however, the full reason for the change is not being communicated. It would be great to have the true "why" of the change. This section should allow the reader/reviewer to get the gist of the related issue without having to open the issue itself.

@minkang3 minkang3 self-requested a review August 21, 2024 19:13
@minkang3
Copy link
Member

ETA: EOD
Availability: 11-5 Weekdays

Copy link
Member

@minkang3 minkang3 left a comment

Choose a reason for hiding this comment

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

Everything looks good except the "Why did you make the changes" section in your PR.

The language "We needed..." doesn't show a clear understanding of the issue and why it needed to be fixed. Rather than saying you did the changes because you needed to, look for the underlying problem that is being fixed.

@izma-mujeeb izma-mujeeb changed the title Added Github Handle Add Github Handle for Nino Dzotsenidze in tech-work-experience.md Aug 21, 2024
Copy link
Member

@minkang3 minkang3 left a comment

Choose a reason for hiding this comment

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

Looks good!

Copy link
Member

@8alpreet 8alpreet left a comment

Choose a reason for hiding this comment

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

Thank you for applying the feedback!

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.

Awesome!

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 Nino Dzotsenidze in tech-work-experience.md

6 participants