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 CONTRIBUTING.md Section 2.7.f #6185

Closed
4 tasks done
elliot-d-kim opened this issue Jan 26, 2024 · 5 comments · Fixed by #6974
Closed
4 tasks done

Update CONTRIBUTING.md Section 2.7.f #6185

elliot-d-kim opened this issue Jan 26, 2024 · 5 comments · Fixed by #6974
Assignees
Labels
Complexity: Small Take this type of issues after the successful merge of your second good first issue Feature: Onboarding/Contributing.md Feature: Wiki 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

Comments

@elliot-d-kim
Copy link
Member

elliot-d-kim commented Jan 26, 2024

Prerequisite

  1. Be a member of Hack for LA. (There are no fees to join.) If you have not joined yet, please follow the steps on our Getting Started page and attend an onboarding session.
  2. Before you claim or start working on an issue, please make sure you have read our How to Contribute to Hack for LA Guide.

Overview

In 2.7.f Working on an issue (6): Handling SSH authorization errors, we need to clarify the instructions so new contributors with existing SSH keys know to skip the entire section.

Details

Hack for LA sees many beginner developers join who may have already created SSH keys on previous beginner projects without necessarily learning what SSH keys are and whether new keys are required for new projects or if existing ones suffice. Thus, instructions should be more explicit that those with SSH keys already connected to their GitHub should skip 2.7.f.

Action Items

  • In your local IDE, navigate to CONTRIBUTING.md
  • Replace
#### **2.7.f Working on an issue (6): Handling SSH authorization errors**

**IMPORTANT:** If you are a windows user then you have to install git bash, but if you are a Mac user you only need to use your terminal. Download git bash for windows here
[Git Bash](https://git-scm.com/downloads)

with

#### **2.7.f Working on an issue (6): Handling SSH authorization errors**

If you already have an SSH key that connects your computer with your GitHub account, you can skip this section (2.7.f).

**IMPORTANT:** If you are a Windows user then you have to install git bash, but if you are a Mac user you only need to use your terminal. Download Git Bash for Windows here:
[Git Bash](https://git-scm.com/downloads)
  • Changes to CONTRIBUTING.md cannot be tested locally, rather they must be tested after pushing the issue branch to your fork of the repository. Push your issue branch in the usual manner, but before creating the Pull Request, check your updates using this test URL. Also store the test URL for use in a later step:
https://github.com/[REPLACE WITH GITHUB HANDLE]/website/blob/[REPLACE WITH NAME OF ISSUE BRANCH]/CONTRIBUTING.md

(for example: https://github.com/bonniewolfe/website/blob/issue-branch-1234/CONTRIBUTING.md)

  • Create a pull request with your changes. In the Pull Request, after the "Why did you make the changes" section, add this line to help reviewers, replacing the text in brackets (and the brackets) with the test URL from the previous Action Item.
For Reviewers: Do not review changes locally, rather, review changes at [REPLACE WITH TEST URL]

Resources/Instructions

CONTRIBUTING.md
Getting started with writing and formatting on GitHub

@elliot-d-kim elliot-d-kim added role: front end Tasks for front end developers role: back end/devOps Tasks for back-end developers Feature: Wiki Complexity: Small Take this type of issues after the successful merge of your second good first issue size: 0.5pt Can be done in 3 hours or less labels Jan 26, 2024
@anilstha1

This comment was marked as outdated.

@elliot-d-kim elliot-d-kim added the ready for dev lead Issues that tech leads or merge team members need to follow up on label Feb 2, 2024
@ExperimentsInHonesty ExperimentsInHonesty added this to the 08. Team workflow milestone Feb 8, 2024
@ExperimentsInHonesty ExperimentsInHonesty added ready for merge team needs a senior review either to do some re writing or to approve it for ready for prioritization and removed ready for dev lead Issues that tech leads or merge team members need to follow up on labels May 7, 2024
@roslynwythe roslynwythe added Ready for Prioritization and removed ready for merge team needs a senior review either to do some re writing or to approve it for ready for prioritization labels May 9, 2024
@andyphancode andyphancode self-assigned this Jun 6, 2024
Copy link

github-actions bot commented Jun 6, 2024

Hi @andyphancode, 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 :)

@andyphancode
Copy link
Member

Availability for this week: Throughout week
My estimated ETA for completing this issue: By 6/7/24 EOD

@ExperimentsInHonesty
Copy link
Member

@andyphancode I have moved this issue to the In Progress column for you. Please remember to do that in future when you assign an issue. Thanks!

@andyphancode
Copy link
Member

@andyphancode I have moved this issue to the In Progress column for you. Please remember to do that in future when you assign an issue. Thanks!

Thank you! @ExperimentsInHonesty

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Complexity: Small Take this type of issues after the successful merge of your second good first issue Feature: Onboarding/Contributing.md Feature: Wiki 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
Development

Successfully merging a pull request may close this issue.

5 participants