Skip to content

added github handle to Maxwell Countryman Skewes profile 6180#6438

Merged
t-will-gillis merged 2 commits intohackforla:gh-pagesfrom
aswutmaxcy:add-github-maxwell-countryman-6180
Mar 11, 2024
Merged

added github handle to Maxwell Countryman Skewes profile 6180#6438
t-will-gillis merged 2 commits intohackforla:gh-pagesfrom
aswutmaxcy:add-github-maxwell-countryman-6180

Conversation

@aswutmaxcy
Copy link
Member

@aswutmaxcy aswutmaxcy commented Mar 7, 2024

Fixes #6180

What changes did you make?

  • added "github-handle"...

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

  • to allow for a value for a github handle to be added in the future

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

No visual changes, just a place to add a github handle in future

@github-actions
Copy link

github-actions bot commented Mar 7, 2024

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 aswutmaxcy-add-github-maxwell-countryman-6180 gh-pages
git pull https://github.com/aswutmaxcy/website.git add-github-maxwell-countryman-6180

Note that CONTRIBUTING.md cannot previewed locally; rather it should be previewed at this URL:

https://github.com/aswutmaxcy/website/blob/add-github-maxwell-countryman-6180/CONTRIBUTING.md  

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

@heejung-hong heejung-hong left a comment

Choose a reason for hiding this comment

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

Hi @aswutmaxcy ,
"github-handle" is spelled incorrectly. The pushed code has "githube".
Please make the corrections and push your code again.
Thank you

edited spelling error on line 92 from "githube-handle:" to "github-handle:"
@aswutmaxcy
Copy link
Member Author

@heejung-hong Thank you for the catch! Just edited the line with the correction.

Copy link
Member

@heejung-hong heejung-hong left a comment

Choose a reason for hiding this comment

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

Good work!
The branches are correct and the issue is linked.
The correct file was changed with the correct information.

Copy link
Member

@marioantonini marioantonini left a comment

Choose a reason for hiding this comment

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

Changes looks good.
Thanks for taking the time to work on this!

@t-will-gillis t-will-gillis self-requested a review March 10, 2024 21:46
Copy link
Member

@t-will-gillis t-will-gillis left a comment

Choose a reason for hiding this comment

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

Hi @aswutmaxcy - everything looks great with your issue, and thanks for working with @heejung-hong to resolve the changes requested.

I have one more request and it involves the formatting for Markdown in the description section: please edit the text for 'what changes were made and why' by adding a space between the dash and the text explanation, and also by removing the extra dash (in two locations). ie replace:

-added "github-handle"...
-

with:

- added "github-handle"...  

Thanks! After that, this PR is ready to merge

@aswutmaxcy
Copy link
Member Author

@t-will-gillis changed the markdown description to copy your comment

@t-will-gillis t-will-gillis self-requested a review March 11, 2024 20:05
Copy link
Member

@t-will-gillis t-will-gillis left a comment

Choose a reason for hiding this comment

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

Looks great, thanks @aswutmaxcy!

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

None yet

Development

Successfully merging this pull request may close these issues.

Add github-handle for Maxwell Countryman Skewes in civic-tech-index.md

4 participants