Skip to content

Conversation

@codyoss
Copy link
Member

@codyoss codyoss commented Jan 5, 2021

Changing the commit formatting so that it can be parsed as a footer
with a conventional commits parser. Here is an example of what a
commit looked like previously:

And what it will look like now:

docs(dialogflow): update comments on parameters and validation result.
PiperOrigin-RevId: 348673154
Source-Link: googleapis/googleapis@0795e3f

Changing the commit formatting so that it can be parsed as a footer
with a conventional commits parser. Here is an example of what a
commit looked like previously:

  - docs(dialogflow): update comments on parameters and validation result.

  PiperOrigin-RevId: 348673154
  Source-Link: googleapis/googleapis@0795e3f

And what it will look like now:

  docs(dialogflow): update comments on parameters and validation result.
    PiperOrigin-RevId: 348673154
    Source-Link: googleapis/googleapis@0795e3f
@codyoss codyoss requested a review from a team as a code owner January 5, 2021 22:48
@google-cla google-cla bot added the cla: yes This human has signed the Contributor License Agreement. label Jan 5, 2021
@codyoss codyoss added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Jan 5, 2021
@codyoss
Copy link
Member Author

codyoss commented Jan 5, 2021

cc @bcoe . Will wait to merge this until the parser work is done.

Copy link
Contributor

@tbpg tbpg left a comment

Choose a reason for hiding this comment

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

High level question: looking at https://www.conventionalcommits.org/en/v1.0.0/, it seems like there should be an empty line between the first line and the next?

@codyoss
Copy link
Member Author

codyoss commented Jan 7, 2021

Related PR googleapis/release-please#686

@bcoe
Copy link

bcoe commented Jan 7, 2021

High level question: looking at https://www.conventionalcommits.org/en/v1.0.0/, it seems like there should be an empty line between the first line and the next?

@tbpg as we formalize the grammar, I've been advocating that we allow single newlines, to protect against human error.

@codyoss codyoss removed the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Jan 11, 2021
@codyoss codyoss requested a review from tbpg January 11, 2021 16:02
@codyoss codyoss added the automerge Merge the pull request once unit tests and other checks pass. label Jan 11, 2021
@codyoss codyoss merged commit d1e3d46 into googleapis:master Jan 11, 2021
@gcf-merge-on-green gcf-merge-on-green bot removed the automerge Merge the pull request once unit tests and other checks pass. label Jan 11, 2021
@codyoss codyoss deleted the cc-footer branch January 11, 2021 23:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants