Skip to content

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
- 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