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
Update CONTRIBUTING.md to require a bug. #211
Conversation
2dd72b8
to
c874d22
Compare
LP: #1532195 Signed-off-by: Kyle Fazzari <kyle@canonical.com>
c874d22
to
b16d917
Compare
|
||
3. Squash commits into one, well-formatted commit. | ||
3. Squash commits into one, well-formatted commit. Mention the bug being | ||
resolved in the commit message on a line all by itself like `LP: #<bug>`. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Does it work also making it a link?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You mean only in the PR instead of the commit? If so, no-- the tools we use for changelog generation parse the commit logs, not the PR descriptions.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you mean adding a link to the commit, that also doesn't work since git-dch parses for the LP: #<bug>
pattern, which a link wouldn't match.
👍 |
👍 a link in the PR like @kyrofa does is an added bonus, not sure if necessary, |
Agreed. Let's merge this, and if the lack of links becomes painful we can revamp. |
Update CONTRIBUTING.md to require a bug.
Improved logic to find non-lo interfaces
Update CONTRIBUTING.md to require a bug.
This PR resolves LP: #1532195 by making the first step of the contribution guidelines "Make sure there's a bug."