Skip to content
This repository was archived by the owner on Nov 8, 2018. It is now read-only.

Fix on Readme for Concourse links #191

Merged
merged 1 commit into from
Jun 25, 2018
Merged

Fix on Readme for Concourse links #191

merged 1 commit into from
Jun 25, 2018

Conversation

brunoban
Copy link
Contributor

How to contribute to the pullrequest resource

Fixes #190

Did you find a bug?

  • Ensure the bug was not already reported by searching on GitHub under Issues.

  • If you're unable to find an open issue addressing the problem, open a new one. Be sure to include a title and clear description, as much relevant information as possible, and a code sample or an executable test case demonstrating the expected behavior that is not occurring.

Did you write a patch that fixes a bug?

  • Open a new GitHub pull request with the patch.

  • Ensure the PR description clearly describes the problem and solution. Include the relevant issue number if applicable.

  • Write a test for your feature. Run all the tests! This will give you and us a higher confidence that nothing broke.

Do you intend to add a new feature or change an existing one?

**Requirements for a Pull Request

A pull request won't be reviewed without the following:

  • Updated docs
  • Test coverage of the added feature
  • Clear explaination of the intention of the pull request.

Thanks!

@crstamps2
Copy link
Collaborator

Thanks for that! Approved!

@crstamps2 crstamps2 merged commit 37d226d into jtarchie:master Jun 25, 2018
@brunoban brunoban deleted the fix-readme branch June 25, 2018 15:23
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Readme is pointing to wrong links on Concourse Docs
2 participants