Skip to content

Conversation

@bamiwoaluko
Copy link
Contributor

initial link on how to contribute to an open project led to a 404 page. I replaced it with a link to lead to the correct pages

Copy link
Contributor

@swzCuroverse swzCuroverse left a comment

Choose a reason for hiding this comment

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

https://docs.github.com/en/get-started/quickstart/contributing-to-projects --> did we want this instead since it is off of the main doc fro github?

@bamiwoaluko
Copy link
Contributor Author

yes. i think it's better to use that link!

@kinow
Copy link
Member

kinow commented Oct 11, 2022

yes. i think it's better to use that link!

I agree too that the GitHub link would be better. @bamiwoaluko can you update the pull request, please?

@bamiwoaluko
Copy link
Contributor Author

yes. i think it's better to use that link!

I agree too that the GitHub link would be better. @bamiwoaluko can you update the pull request, please?

will do!

Copy link
Member

@kinow kinow left a comment

Choose a reason for hiding this comment

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

@bamiwoaluko look at the very end of this document. There is a line like this one [how-contribute]: ..... That's where we should update it 😉

## Using GitHub

If you choose to contribute via GitHub, you may want to look at
[How to Contribute to an Open Source Project on GitHub][how-contribute].
Copy link
Member

Choose a reason for hiding this comment

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

][how-contribute].... this is an alias for a link. The actual link is at the end of this document, along with several other links. That's useful in case we need to use the link in multiple places (but update a single place) and also to reduce cluttering in the source.

kinow
kinow previously approved these changes Oct 11, 2022
Copy link
Member

@kinow kinow 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 to me! @bamiwoaluko are you familiar with git and squashing commits? If so you could perhaps try to squash the commits in this PR as there are now 4 commits, but for a small change like this one would be better to have a single commit (easier for others to compare changes later if needed). If you are not sure how to do it, a reviewer can squash it for you when they are ready to merge it 👍 Thanks!

mr-c
mr-c previously approved these changes Oct 13, 2022
initial link on how to contribute to an open project led to a 404 page. I replaced it with a link to lead to the correct pages
@mr-c mr-c merged commit 990bc06 into common-workflow-language:main Oct 13, 2022
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.

4 participants