Skip to content

Conversation

fanquake
Copy link
Member

Update the developer docs after the recent subtree renaming.

@fanquake fanquake added the Docs label Sep 29, 2021
@laanwj
Copy link
Member

laanwj commented Sep 29, 2021

ACK 7255125

Copy link
Member

@maflcko maflcko left a comment

Choose a reason for hiding this comment

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

Why not update test/lint/README.md ?

@fanquake fanquake force-pushed the document_subtree_renaming branch from 7255125 to 2b90eae Compare September 29, 2021 07:12
@fanquake
Copy link
Member Author

Why not update test/lint/README.md ?

I forgot we duplicate this stuff all over the place. Have updated.

Copy link
Member

@maflcko maflcko left a comment

Choose a reason for hiding this comment

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

ACK

@laanwj
Copy link
Member

laanwj commented Sep 29, 2021

I forgot we duplicate this stuff all over the place. Have updated.

We don't need to, could put it in one place then refer to it.
But this works, re-ACK.

@maflcko maflcko mentioned this pull request Sep 29, 2021
Copy link
Member

@hebasto hebasto left a comment

Choose a reason for hiding this comment

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

ACK 2b90eae

@DrahtBot
Copy link
Contributor

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

Conflicts

Reviewers, this pull request conflicts with the following ones:

  • #23114 (Add minisketch subtree and integrate into build/test by fanquake)
  • #21515 (Erlay: bandwidth-efficient transaction relay protocol by naumenkogs)

If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

@fanquake fanquake merged commit 69a66dc into bitcoin:master Sep 30, 2021
@fanquake fanquake deleted the document_subtree_renaming branch September 30, 2021 00:53
sidhujag pushed a commit to syscoin/syscoin that referenced this pull request Sep 30, 2021
@bitcoin bitcoin deleted a comment from tieorange Oct 21, 2021
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Oct 21, 2021
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.

5 participants