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

Updated contributing guide for GitHub issues #2046

Merged
merged 1 commit into from
May 1, 2018

Conversation

SkyeHoefling
Copy link
Contributor

@SkyeHoefling SkyeHoefling commented May 1, 2018

Updated contribution guide to use GitHub Issues instead of JIRA issues

#2044

@SkyeHoefling SkyeHoefling changed the title #2044 Updatec contributing guide for GitHub issues #2044 Updated contributing guide for GitHub issues May 1, 2018
@bdukes
Copy link
Contributor

bdukes commented May 1, 2018

I think we just need the issue number in the body of the PR (and, optionally, body of the commit message), having it in the PR/commit title doesn't seem to make a difference. For example

Make the example in CONTRIBUTING imperative and concrete

Fixes #2044

@SkyeHoefling
Copy link
Contributor Author

@bdukes I agree it is unnecessary in the title. Does commit 4aa5cb3 cover it?

@SkyeHoefling SkyeHoefling changed the title #2044 Updated contributing guide for GitHub issues Updated contributing guide for GitHub issues May 1, 2018
@SkyeHoefling
Copy link
Contributor Author

I think I am going to clean this up a little based on your feedback

@bdukes
Copy link
Contributor

bdukes commented May 1, 2018

@ahoefling I pushed a commit with a change more like what I'm thinking (feel free to adjust how you see fit)

@SkyeHoefling
Copy link
Contributor Author

@bdukes I didn't see your message and I squashed the commits. Can you reapply it? Sorry about that, should have checked here before I squashed.

@bdukes
Copy link
Contributor

bdukes commented May 1, 2018

@ahoefling how's that look?

@SkyeHoefling
Copy link
Contributor Author

Looks good to me

…ue ID in the PR and no longer needing the jira ID in the commit
@bdukes bdukes merged commit f892701 into dnnsoftware:development May 1, 2018
@ohine ohine added this to the 9.2.1 milestone Jun 21, 2018
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.

3 participants