Skip to content

wrap u8 and LeafVersion in backticks and square bracket in doc#766

Merged
dr-orlovsky merged 1 commit intorust-bitcoin:masterfrom
kafaichoi:add-backticks-and-square-bracket-to-u8-and-LockVersion-in-comment
Jan 10, 2022
Merged

wrap u8 and LeafVersion in backticks and square bracket in doc#766
dr-orlovsky merged 1 commit intorust-bitcoin:masterfrom
kafaichoi:add-backticks-and-square-bracket-to-u8-and-LockVersion-in-comment

Conversation

@kafaichoi
Copy link
Copy Markdown
Contributor

Found this minor doc issue while reviewing(learning) previous merged PR.

Close #763

@dr-orlovsky dr-orlovsky added documentation trivial Obvious, easy and quick to review (few lines or doc-only...) labels Jan 10, 2022
Copy link
Copy Markdown
Collaborator

@dr-orlovsky dr-orlovsky left a comment

Choose a reason for hiding this comment

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

ACK 9835736

@Kixunil
Copy link
Copy Markdown
Collaborator

Kixunil commented Jan 10, 2022

ACK 9835736

thanks!

@dr-orlovsky
Copy link
Copy Markdown
Collaborator

@Kixunil you haven't actually used "Approve" GitHub button and this PR is still unmergable because of that

Copy link
Copy Markdown
Collaborator

@Kixunil Kixunil left a comment

Choose a reason for hiding this comment

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

ACK 9835736

@Kixunil
Copy link
Copy Markdown
Collaborator

Kixunil commented Jan 10, 2022

Should've drink coffee before work :D

@dr-orlovsky dr-orlovsky merged commit b91058c into rust-bitcoin:master Jan 10, 2022
@apoelstra
Copy link
Copy Markdown
Member

Does the Bitcoin Core github merge script also insist that you use the "approve" button?

@RCasatta
Copy link
Copy Markdown
Collaborator

Does the Bitcoin Core github merge script also insist that you use the "approve" button?

I can confirm the push is declined if there aren't 2 approve button clicks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

trivial Obvious, easy and quick to review (few lines or doc-only...)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Put backticks/square brackets around ` u8 and [LeafVersion]`

5 participants