Skip to content

copy AGENTS.md, CLAUDE.md, CONTRIBUTING.md and SECURITY.md from rust-bitcoin#962

Open
apoelstra wants to merge 1 commit into
rust-bitcoin:masterfrom
apoelstra:2026-05/policies
Open

copy AGENTS.md, CLAUDE.md, CONTRIBUTING.md and SECURITY.md from rust-bitcoin#962
apoelstra wants to merge 1 commit into
rust-bitcoin:masterfrom
apoelstra:2026-05/policies

Conversation

@apoelstra
Copy link
Copy Markdown
Member

I deleted large chunks of CONTRIBUTING.md that didn't apply, and changed the preamble of SECURITY.md to say it applies to rust-miniscript rather than to the rust-bitcoin core projects.

Otherwise no changes.

Hopefully this will reduce the amount of LLM slop showing up on this repo lately.

I will also prioritize getting forgejo's API integrated into my CI system so that we can just migrate this project off of Github, which should further help.

@apoelstra
Copy link
Copy Markdown
Member Author

cc @sanket1729 you might want to take a look at this one

Copy link
Copy Markdown
Contributor

@trevarj trevarj left a comment

Choose a reason for hiding this comment

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

Concept ACK 78ec9a0

Comment thread CONTRIBUTING.md Outdated
to an open source project or pick up Rust while contributing. Don't be shy,
you'll learn.

For a more in depth discussion of our coding policy see [policy.md](./docs/policy.md)
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

broken link

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Huh, I was sure I deleted this paragraph.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Ah, no, it was the # Policy section I deleted. I'll also delete this line.

Copy link
Copy Markdown
Member

@sanket1729 sanket1729 left a comment

Choose a reason for hiding this comment

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

Looks good to me.

ACK 78ec9a0

Comment thread CONTRIBUTING.md Outdated
@@ -0,0 +1,282 @@
# Contributing to rust-bitcoin
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
# Contributing to rust-bitcoin
# Contributing to rust-miniscript

Comment thread CONTRIBUTING.md Outdated
Comment on lines +5 to +7
The following is a set of guidelines for contributing to Rust Bitcoin
implementation and other Rust Bitcoin-related projects, which are hosted in the
[Rust Bitcoin Community](https://github.com/rust-bitcoin) on GitHub. These are
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Should we just link to the CONTRIBUTING file in rust-bitcoin?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Yeah, that's a good idea.

@tcharding
Copy link
Copy Markdown
Member

ACK everything except CONTRIBUTING.md

…bitcoin

I deleted large chunks of CONTRIBUTING.md that didn't apply, and changed the
preamble of SECURITY.md to say it applies to rust-miniscript rather than to
the rust-bitcoin core projects.

Otherwise no changes.
@apoelstra
Copy link
Copy Markdown
Member Author

Updated to change a few more instances of "bitcoin" to "miniscript", and to add a link to the rust-bitcoin CONTRIBUTING.md doc.

Copy link
Copy Markdown
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 46423ba

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.

5 participants