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

Move all example pallets under examples folder. #10215

Merged
merged 7 commits into from
Nov 10, 2021

Conversation

jimmychu0807
Copy link
Contributor

This should be a pretty trivial update.

Signed-off-by: Jimmy Chu <jimmychu0807@gmail.com>
Signed-off-by: Jimmy Chu <jimmychu0807@gmail.com>
@jimmychu0807 jimmychu0807 requested a review from a team as a code owner November 8, 2021 11:53
@github-actions github-actions bot added the A0-please_review Pull request needs code review. label Nov 8, 2021
@jimmychu0807 jimmychu0807 added 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. labels Nov 8, 2021
Signed-off-by: Jimmy Chu <jimmychu0807@gmail.com>
@jimmychu0807 jimmychu0807 added the D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit label Nov 8, 2021
Signed-off-by: Jimmy Chu <jimmychu0807@gmail.com>
"frame/example-parallel",
"frame/examples/basic",
"frame/examples/offchain-worker",
"frame/examples/parallel",
Copy link
Member

Choose a reason for hiding this comment

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

Can we add some information that this is highly experimental and not really supported? Maybe in the crate level docs?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I updated with a note that these example pallets are not meant to be used in production in this commit.

@jimmychu0807
Copy link
Contributor Author

I don't have right to merge the pull request and will need one of you to merge this PR @bkchr @apopiak @TriplEight . Thanks.

@bkchr
Copy link
Member

bkchr commented Nov 10, 2021

bot merge

@paritytech-processbot paritytech-processbot bot merged commit 7b24a94 into master Nov 10, 2021
@paritytech-processbot paritytech-processbot bot deleted the jc/move-examples branch November 10, 2021 08:11
@nuke-web3
Copy link
Contributor

Like it! Although @jimmychu0807 you broke doc links FYI - we should be aware that moving things around might do this in docs - the devhub CI will quickly spot it, but as a nice-to-have please consider this in future PRs.

@nuke-web3
Copy link
Contributor

dumb q: are these pallets tested in CI at this level of the directory structure?

ordian added a commit that referenced this pull request Nov 12, 2021
* master: (27 commits)
  Bump rustversion from 1.0.4 to 1.0.5 (#10243)
  Kill the light client, CHTs and change tries. (#10080)
  tuple to struct event variants (#10206)
  Bump thiserror from 1.0.26 to 1.0.30 (#10240)
  Warn about usage of pallet collective set members call. (#10156)
  Bump git2 from 0.13.22 to 0.13.23 (#10238)
  Add group name in task metrics  (#10196)
  Bump proc-macro-crate from 1.0.0 to 1.1.0 (#10237)
  Bump parity-util-mem from 0.10.0 to 0.10.2 (#10236)
  Bump substrate-bip39 from 0.4.2 to 0.4.4 (#10213)
  Upgrade jsonrpsee to v0.4.1 (#10022)
  expose substrate-cli service (#10229)
  Intend to reactivate cargo-unleash check (#10167)
  CI: build docs with deps (#9884)
  use CountedMap in pallet-bags-list (#10179)
  Move all example pallets under `examples` folder. (#10215)
  Upgrade wasm builder (#10226)
  upgrade ss58-registry with additional networks. (#10224)
  move wiki -> docs (#10225)
  new remote-ext mode: (#10192)
  ...
@bkchr
Copy link
Member

bkchr commented Nov 12, 2021

dumb q: are these pallets tested in CI at this level of the directory structure?

Yes: https://github.com/paritytech/substrate/pull/10215/files#diff-037ea159eb0a7cb0ac23b851e66bee30fb838ee8d0d99fa331a1ba65283d37f7L485

grishasobol pushed a commit to gear-tech/substrate that referenced this pull request Mar 28, 2022
* Put all examples under one folder

Signed-off-by: Jimmy Chu <jimmychu0807@gmail.com>

* Updated Cargo.toml

Signed-off-by: Jimmy Chu <jimmychu0807@gmail.com>

* updated for ci script

Signed-off-by: Jimmy Chu <jimmychu0807@gmail.com>

* update

Signed-off-by: Jimmy Chu <jimmychu0807@gmail.com>

* Added notes that example pallets are not meant to be used in production.

Signed-off-by: Jimmy Chu <jimmychu0807@gmail.com>

* updated

Signed-off-by: Jimmy Chu <jimmychu0807@gmail.com>
ark0f pushed a commit to gear-tech/substrate that referenced this pull request Feb 27, 2023
* Put all examples under one folder

Signed-off-by: Jimmy Chu <jimmychu0807@gmail.com>

* Updated Cargo.toml

Signed-off-by: Jimmy Chu <jimmychu0807@gmail.com>

* updated for ci script

Signed-off-by: Jimmy Chu <jimmychu0807@gmail.com>

* update

Signed-off-by: Jimmy Chu <jimmychu0807@gmail.com>

* Added notes that example pallets are not meant to be used in production.

Signed-off-by: Jimmy Chu <jimmychu0807@gmail.com>

* updated

Signed-off-by: Jimmy Chu <jimmychu0807@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. 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.

5 participants