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

improve instructions in CONTRIBUTING.md #36082

Merged
merged 4 commits into from
Jan 24, 2019

Conversation

DanGrayson
Copy link
Contributor

See the discussion at #35920.

Copy link
Member

@SMillerDev SMillerDev left a comment

Choose a reason for hiding this comment

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

I think this is already better than before but I think there's still some improvements to be made.

CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Show resolved Hide resolved
@DanGrayson
Copy link
Contributor Author

Thanks for the feedback, now take a look.

@SMillerDev
Copy link
Member

Looks good to me.

@DanGrayson
Copy link
Contributor Author

Thanks. Perhaps you could mark your review so it shows the changes are now accepted.

CONTRIBUTING.md Outdated Show resolved Hide resolved
@DanGrayson
Copy link
Contributor Author

DanGrayson commented Jan 18, 2019 via email

@claui
Copy link
Contributor

claui commented Jan 18, 2019

Thanks @DanGrayson! Apologies for the confusion.

@SMillerDev
Copy link
Member

@claui ready to merge?

@SMillerDev SMillerDev added the almost there PR is nearly ready to merge label Jan 24, 2019
@claui
Copy link
Contributor

claui commented Jan 24, 2019

@SMillerDev I made a change in order to clarify the commit messages. Mind reviewing my change one last time before we merge?

@SMillerDev
Copy link
Member

Looks good to me.

@DanGrayson
Copy link
Contributor Author

Fine with me.

@claui claui merged commit 23fc5ac into Homebrew:master Jan 24, 2019
@claui
Copy link
Contributor

claui commented Jan 24, 2019

Thanks for your contribution @DanGrayson! 🍻

@claui claui removed the almost there PR is nearly ready to merge label Jan 24, 2019
@lock lock bot added the outdated PR was locked due to age label Feb 23, 2019
@lock lock bot locked as resolved and limited conversation to collaborators Feb 23, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
outdated PR was locked due to age
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants