-
Notifications
You must be signed in to change notification settings - Fork 380
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I would be fine to move this to polkadot-parachains/runtime/*
, but not moving it to the top
That works for me. The most important is that:
It is easier if the Let me adjust to your suggestion. |
I don't see a benefit going from |
@bkchr would that work for you now ? |
bot merge |
Trying merge. |
Merge failed with response status: 403 Forbidden and body: |
chore: refactor for consistant naming of the crates and paths of the runtimes * tests: ignore folder created by the tests * chore: refactoring * Update polkadot-parachains/Cargo.toml Co-authored-by: Bastian Köcher <bkchr@users.noreply.github.com>
This PR renames the rococo runtime to follow the pattern used by all other chains (in this repo and the pollkadot repo).
The PR also uses the same folder tree (= runtimes under a
runtime/<name>
) folder. That simplifies tooling and allows building all the runtimes the same way.Once this PR is merged, I will be able to extend the CI to build statemint, shell and rococo using srtool (see #509)