Skip to content

Conversation

@maflcko
Copy link
Member

@maflcko maflcko commented Jul 20, 2021

Seems to be the most common template used, so adjust this here, too.

Copy link
Member

@hebasto hebasto left a comment

Choose a reason for hiding this comment

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

ACK fa3b92d20570c72a3178c693afb81e105849805f.

Curly braces could be skipped for clarity.

@maflcko
Copy link
Member Author

maflcko commented Jul 20, 2021

I like to always add them for clarity

@DrahtBot DrahtBot added the Docs label Jul 20, 2021
@maflcko maflcko force-pushed the 2107-docGuixCommit branch from fa3b92d to fafade9 Compare July 20, 2021 15:50
Copy link
Member

@hebasto hebasto left a comment

Choose a reason for hiding this comment

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

re-ACK fafade9

@laanwj
Copy link
Member

laanwj commented Jul 20, 2021

ACK fafade9

@laanwj laanwj merged commit bb09ec6 into bitcoin:master Jul 20, 2021
@maflcko maflcko deleted the 2107-docGuixCommit branch July 21, 2021 14:13
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Jul 23, 2021
….sigs repo

fafade9 doc: Adjust commit message template for the guix.sigs repo (MarcoFalke)

Pull request description:

  Seems to be the most common template used, so adjust this here, too.

ACKs for top commit:
  laanwj:
    ACK fafade9
  hebasto:
    re-ACK fafade9

Tree-SHA512: 20477d14ecfad94f3b28b94786a4c0d98df539360d0c1deefa94766064a7d0700c849e54d6b251f922e135fcfa964ada0c724090f7f92b459ea39f7c3ca8c65d
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Aug 16, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants