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

Example pull request #36

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Example pull request #36

wants to merge 2 commits into from

Conversation

rainsworth
Copy link
Owner

This pull request adds an example markdown file and will serve as an example pull request for the demonstration.

Pull requests are proposed changes to a repository submitted by a user and accepted or rejected by a repository's owner (or designated collaborators). Like issues, pull requests each have their own discussion forum.

It’s important to provide as much useful information and a rationale for why you’re making this pull request in the first place. The project owner needs to be able to determine whether your change is as useful to everyone as you think it is.

It's helpful to summarise the changes in a list:

  • Create and add content to example.md
  • Any other changes I made that are included in this pull request

You can also tag related issues using a hashtag: #18 or link to them in the right sidebar.

@rainsworth
Copy link
Owner Author

Thank you for your contribution! 🎉

We will review your pull request and get back to you as soon as possible.

@rainsworth
Copy link
Owner Author

Hi @rainsworth, this is really valuable 👍 I just have a few suggestions for minor changes before we merge this into the main branch. Can you please fix the following:

  • Add a list
  • Add a link
  • Bold some text

@rainsworth rainsworth added the documentation Improvements or additions to documentation label Jun 30, 2020
@rainsworth
Copy link
Owner Author

Thanks @rainsworth, I have now made those changes and also removed some text because of a reason.

@rainsworth
Copy link
Owner Author

Brilliant, I will go ahead and merge this now!

@rainsworth rainsworth mentioned this pull request Jun 30, 2020
3 tasks
@rainsworth rainsworth added this to In progress in Example project Jun 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
Example project
  
In progress
Development

Successfully merging this pull request may close these issues.

None yet

1 participant