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

doc: Update contributor guidelines link in README.md #238

Merged
merged 2 commits into from
Sep 3, 2024

Conversation

nwekealex65
Copy link
Contributor

Issue Resolved
This pull request resolve issue #237, where the contributor guidelines link in the README.md file was leading to a 404 error page.

Changes Made:

Updated the link in the README.md file to correctly point to the location of the contributor guidelines.
Ensured that the link redirects properly and allows contributors to access the guidelines without encountering any errors.

Impact:

Provides clarity and accessibility for contributors seeking guidance on project contributions.
Removes a barrier to entry for potential contributors, fostering a more inclusive and collaborative environment.
Improves overall project documentation and usability.

Testing Done:

Tested the updated link to verify that it redirects to the correct location without any errors.

Fixes the broken link leading to a 404 error page. Updated the link to point to the correct location of the contributor guidelines to ensure contributors can access them properly.
Readme.md Outdated
@@ -53,4 +53,4 @@ Checkout Docker Installation [docs](./docs/Docker-installation.md)
## Get involved!

* Start by reading our [Code of conduct](https://github.com/hotosm/fAIr/blob/master/docs/Code-of-Conduct.md)
* Get familiar with our [contributor guidelines](https://github.com/hotosm/fAIr/blob/master/docs/Contribution.md) explaining the different ways in which you can support this project! We need your help!
* Get familiar with our [contributor guidelines](https://github.com/hotosm/fAIr/blob/master/CONTRIBUTING.md) explaining the different ways in which you can support this project! We need your help!
Copy link
Member

Choose a reason for hiding this comment

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

Use relative file path , So that it will work on any branch and fork

Copy link
Contributor Author

Choose a reason for hiding this comment

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

okay @kshitijrajsharma on it now. Will let you know when its done

@@ -53,4 +53,4 @@ Checkout Docker Installation [docs](./docs/Docker-installation.md)
## Get involved!

* Start by reading our [Code of conduct](https://github.com/hotosm/fAIr/blob/master/docs/Code-of-Conduct.md)
* Get familiar with our [contributor guidelines](https://github.com/hotosm/fAIr/blob/master/docs/Contribution.md) explaining the different ways in which you can support this project! We need your help!
* Get familiar with our [contributor guidelines](CONTRIBUTING.md) explaining the different ways in which you can support this project! We need your help!
Copy link
Member

Choose a reason for hiding this comment

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

Use relative path ./

@kshitijrajsharma kshitijrajsharma merged commit 3554d7c into hotosm:master Sep 3, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants