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

Fix small typos in ci README #15

Merged
merged 1 commit into from
Nov 7, 2024
Merged

Conversation

raphjaph
Copy link
Contributor

@raphjaph raphjaph commented Sep 9, 2024

I was going through the README trying to add the rust-bitcoin CI to my BIP-322 crate and found some small things I needed to change to make it work for me.

Copy link
Member

@apoelstra apoelstra left a comment

Choose a reason for hiding this comment

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

utACK 39a0b85

Copy link
Member

@apoelstra apoelstra left a comment

Choose a reason for hiding this comment

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

utACK 39a0b85

@apoelstra
Copy link
Member

Interesting. I definitely only ran the gh pr review command once.

ci/README.md Outdated
@@ -195,7 +195,7 @@ jobs:
uses: actions/checkout@v4
- name: Read nightly version
id: read_toolchain
run: echo "nightly_version=$(cat nightly-version)" >> $GITHUB_OUTPUT
run: echo "nightly_version=$(cat nightly_version)" >> $GITHUB_OUTPUT
Copy link
Member

Choose a reason for hiding this comment

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

This is wrong, the file is called nightly-version.

Copy link
Contributor Author

@raphjaph raphjaph Nov 5, 2024

Choose a reason for hiding this comment

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

Ok then line 181 must be wrong. I'll change it.

$ cat nightly_version

@tcharding
Copy link
Member

Thanks man, can you squash into a single commit please.

@raphjaph
Copy link
Contributor Author

raphjaph commented Nov 7, 2024

Thanks man, can you squash into a single commit please.

For sure!

Copy link
Member

@tcharding tcharding left a comment

Choose a reason for hiding this comment

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

ACK b44c53b

@tcharding
Copy link
Member

Thanks man!

Copy link
Member

@apoelstra apoelstra left a comment

Choose a reason for hiding this comment

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

utACK b44c53b

Copy link
Member

@apoelstra apoelstra left a comment

Choose a reason for hiding this comment

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

utACK b44c53b

@apoelstra apoelstra merged commit 9540345 into rust-bitcoin:master Nov 7, 2024
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants