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

psbt: s/transcation/transaction/ #26997

Merged
merged 1 commit into from
Feb 16, 2023
Merged

Conversation

instagibbs
Copy link
Member

No description provided.

@DrahtBot
Copy link
Contributor

DrahtBot commented Jan 30, 2023

The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

Reviews

See the guideline for information on the review process.

Type Reviewers
ACK fanquake

If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

Conflicts

No conflicts as of last run.

@maflcko maflcko changed the title s/transcation/transaction/ psbt: s/transcation/transaction/ Jan 30, 2023
@DrahtBot DrahtBot added the PSBT label Jan 30, 2023
@@ -1164,7 +1164,7 @@ struct PartiallySignedTransaction

// Make sure that we got an unsigned tx
if (!tx) {
throw std::ios_base::failure("No unsigned transcation was provided");
throw std::ios_base::failure("No unsigned transaction was provided");
Copy link
Member

@jonatack jonatack Jan 30, 2023

Choose a reason for hiding this comment

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

maybe also fix up this one that was recently added in 1429f83

$ ./test/lint/lint-spelling.py 
src/blockencodings.h:138: overriden ==> overridden

Copy link
Member

Choose a reason for hiding this comment

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

Another one now, may as well get 'em all in this fixup commit.

$ test/lint/lint-spelling.py 
src/blockencodings.h:138: overriden ==> overridden
src/net.cpp:1668: addressses ==> addresses

Copy link
Member

@fanquake fanquake left a comment

Choose a reason for hiding this comment

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

ACK 9066314 - looks like other comments are being addressed elsewhere.

@fanquake fanquake merged commit 424fc60 into bitcoin:master Feb 16, 2023
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Feb 16, 2023
9066314 s/transcation/transaction/ (Greg Sanders)

Pull request description:

ACKs for top commit:
  fanquake:
    ACK 9066314 - looks like other comments are being addressed elsewhere.

Tree-SHA512: c835a14db2e0cf5e0317c95c8c7441df1f7c6cb14be7809fd947e07ea9d23f1f171f111429aabd0509b7f17601bc742041316b18e1135e547a966961f2c65038
@bitcoin bitcoin locked and limited conversation to collaborators Feb 16, 2024
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