Skip to content

Added hyperlink to Contributing to codeigniter section #845

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

Merged
merged 3 commits into from
Nov 27, 2017

Conversation

vinodselvin
Copy link
Contributor

@vinodselvin vinodselvin commented Nov 27, 2017

Added hyperlink of Contributingmd file
https://github.com/bcit-ci/CodeIgniter4/blob/develop/contributing.md to ReadMe

@vinodselvin
Copy link
Contributor Author

Why updating ReadMe.md file Travis is failing?

@lonnieezell
Copy link
Member

Why updating ReadMe.md file Travis is failing?

If you click on the "Details" link next to the failing Travis job you'll see it's completely un-related to your PR.

README.md Outdated
@@ -63,7 +63,7 @@ if you want to take the lead for one of them.

We are not looking for out-of-scope contributions, only those that would be considered part of our controlled evolution!

Please read the *Contributing to CodeIgniter* section in the user guide
Please read the [*Contributing to CodeIgniter*](contributing.md) section in the user guide
Copy link
Member

Choose a reason for hiding this comment

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

This link won't actually link to the file since they're not in the same folder. Please provide a full URL to the page here on GitHub, so that it will work correctly when viewing on GitHub.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ok will provide complete url of contributing.md

@lonnieezell lonnieezell merged commit 4ca7b49 into codeigniter4:develop Nov 27, 2017
@vinodselvin vinodselvin deleted the patch-1 branch November 27, 2017 14:30
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