Skip to content

Edit content field and remove type field from job.yml#3615

Merged
blulady merged 1 commit intohackforla:gh-pagesfrom
MattPereira:edit-remove-field-2845
Oct 14, 2022
Merged

Edit content field and remove type field from job.yml#3615
blulady merged 1 commit intohackforla:gh-pagesfrom
MattPereira:edit-remove-field-2845

Conversation

@MattPereira
Copy link
Contributor

@MattPereira MattPereira commented Oct 12, 2022

Fixes #2845

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

  • Changed content field key: value pair in _data/internal/credits/job.yml
  • Removed type field from _data/internal/credits/job.yml

These changes were made to reduce redundancy in the code.

Screenshots of Proposed Changes Of The Website

Updating content field and type field of job.yml file. 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 MattPereira-edit-remove-field-2845 gh-pages
git pull https://github.com/MattPereira/website.git edit-remove-field-2845

@github-actions github-actions bot added good first issue Good for newcomers P-Feature: Credit https://www.hackforla.org/credits/ role: back end/devOps Tasks for back-end developers role: front end Tasks for front end developers size: 0.5pt Can be done in 3 hours or less labels Oct 12, 2022
@blulady
Copy link
Member

blulady commented Oct 12, 2022

Hey Matt,
So again I appreciate your voracity but volunteers assign themselves to review PR's. Since this is a good first PR, it would be better if other new devs got a chance to review it. Normally I would recommend reviewing other dev's pr. Right now we don't have much but as other folks make their first pr, you'll have a ton to review.

@GRISONRF GRISONRF self-requested a review October 12, 2022 17:43
@GRISONRF
Copy link
Member

ETA: 10/12/2022
Availability: 35min

Copy link
Member

@GRISONRF GRISONRF left a comment

Choose a reason for hiding this comment

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

The changes were made in the correct branch and everything looks good.

Edit: I just noticed it some checks were not successful and I'm not sure why.
It got an error parsing the pull request body for all the linked issues, but I don't know if it's related to Matt's code, since everything looks right.

@giroz giroz self-requested a review October 12, 2022 19:44
@giroz
Copy link
Member

giroz commented Oct 12, 2022

ETA: 10/13/22
Availability: 2 hours

@arpitapandya
Copy link
Member

The changes were made in the correct branch and everything looks good.

Edit: I just noticed it some checks were not successful and I'm not sure why. It got an error parsing the pull request body for all the linked issues, but I don't know if it's related to Matt's code, since everything looks right.

Answer on why some checks fail: We have GitHub actions that check for linked issue when we create a pull request. I see that @MattPereira mistakenly typed Fixes #3615 while creating a pull request to link the issue so checks didn't pass and that failed to generate PR labels automatically. but, later on, the correct issue is linked.

@gstemmann
Copy link
Member

Review ETA: 6 PM 10/15/22
Availability: 6 hours

Copy link
Member

@gstemmann gstemmann 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 to me. And the correct repositories were chosen.

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.

This branch is coming from the correct branch and going into gh-pages. There is a linked issue, it is correct and addressed in the PR by changing line four to content-type: image and deleting line 11. The code looks clean and there is only one file changed.
Good job!!

@blulady blulady merged commit b1897c7 into hackforla:gh-pages Oct 14, 2022
Copy link
Member

@giroz giroz left a comment

Choose a reason for hiding this comment

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

Good job. line 4 added text, and line 11 deleted text

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: Credit https://www.hackforla.org/credits/ role: back end/devOps Tasks for back-end developers role: front end Tasks for front end developers size: 0.5pt Can be done in 3 hours or less

Projects

Development

Successfully merging this pull request may close these issues.

Edit content field and remove type field from job.yml credits file

6 participants