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

docs(guidelines): improve contribution guidelines #180

Merged
merged 3 commits into from
Jan 25, 2018

Conversation

acatl
Copy link
Collaborator

@acatl acatl commented Jan 24, 2018

change templates + force codemods on breaking changes

What: change github templates format + contribution.md ask developers to send codemods when breaking changes

Why: improve quality of issues and pullrequest

How: changing templates

Checklist:

  • Documentation
  • Tests
  • Ready to be merged
  • Added username to all-contributors list

change templates + force codemods on breaking changes
@acatl acatl added this to the v2.x milestone Jan 24, 2018
@acatl acatl self-assigned this Jan 24, 2018
@coveralls
Copy link

coveralls commented Jan 24, 2018

Coverage Status

Coverage remained the same at 100.0% when pulling 0ba1524 on acatl:github-issue-templates into 8ac4f37 on ViacomInc:master.

CONTRIBUTING.md Outdated
data-point-codemods that will help users upgrade their codebase.

Breaking changes without a codemod will not be accepted unless a codemod is not
viable or does not apply to the specific situation.
Copy link
Collaborator

Choose a reason for hiding this comment

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

A couple of lines have hard line breaks but other longer lines don't. Can we make them consistent?

@@ -0,0 +1,38 @@
const Util = require('util')
Copy link
Collaborator

Choose a reason for hiding this comment

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

This file seems irrelevant to the PR.

Copy link
Collaborator

@paulmolluzzo paulmolluzzo 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 you added a file by accident so requesting change to remove it.

Otherwise this looks good and worth including: 👍

@acatl
Copy link
Collaborator Author

acatl commented Jan 24, 2018

@paulmolluzzo addressed your feedback

@acatl acatl closed this Jan 24, 2018
@acatl acatl reopened this Jan 24, 2018
Copy link
Collaborator

@paulmolluzzo paulmolluzzo left a comment

Choose a reason for hiding this comment

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

Great!

@acatl acatl merged commit a8e98b8 into ViacomInc:master Jan 25, 2018
@acatl acatl deleted the github-issue-templates branch January 25, 2018 14:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants