Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Move grandpa crates to consensus folder #13458

Merged
merged 10 commits into from
Feb 27, 2023

Conversation

davxy
Copy link
Member

@davxy davxy commented Feb 24, 2023

As per title, plus some renaming here and there for coherence.

Renamed crates:

  • sp-finality-grandpa -> sp-consensus-grandpa
  • sc-finality-grandpa -> sc-consensus-grandpa

Partially addresses #13449

polkadot companion: paritytech/polkadot#6776

@davxy davxy self-assigned this Feb 24, 2023
@davxy davxy added the C1-low PR touches the given topic and has a low impact on builders. label Feb 24, 2023
@paritytech-cicd-pr
Copy link

The CI pipeline was cancelled due to failure one of the required jobs.
Job name: cargo-check-benches
Logs: https://gitlab.parity.io/parity/mirrors/substrate/-/jobs/2439521

@davxy davxy added D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit B1-note_worthy Changes should be noted in the release notes A0-pleasereview B0-silent Changes should not be mentioned in any release notes and removed B1-note_worthy Changes should be noted in the release notes labels Feb 24, 2023
@davxy davxy requested a review from a team February 27, 2023 10:58
ukint-vs pushed a commit to gear-tech/substrate that referenced this pull request Apr 10, 2023
* Move grandpa under consensus dir
* Rename grandpa folder
* Finish grandpa renaming
* Minor tweaks
* Cargo fmt
* Adjust path to chain spec
nathanwhit pushed a commit to nathanwhit/substrate that referenced this pull request Jul 19, 2023
* Move grandpa under consensus dir
* Rename grandpa folder
* Finish grandpa renaming
* Minor tweaks
* Cargo fmt
* Adjust path to chain spec
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants