-
Notifications
You must be signed in to change notification settings - Fork 707
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Move developer-hub
to sdk-docs
#2565
Conversation
…k into kiz-developer-hub
…k into kiz-developer-hub
Co-authored-by: Juan Girini <[email protected]>
Co-authored-by: bader y <[email protected]>
Co-authored-by: bader y <[email protected]>
Co-authored-by: Sebastian Kunert <[email protected]>
Co-authored-by: Sebastian Kunert <[email protected]>
Co-authored-by: Sebastian Kunert <[email protected]>
Co-authored-by: Juan Girini <[email protected]> Co-authored-by: Sebastian Kunert <[email protected]> Co-authored-by: Liam Aharon <[email protected]>
Co-authored-by: Liam Aharon <[email protected]> Co-authored-by: Juan Girini <[email protected]> Co-authored-by: Sebastian Kunert <[email protected]>
The CI pipeline was cancelled due to failure one of the required jobs. |
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.
Found some "developer_hub" references, not sure if you planned to do it in a separate PR?
I think it might be good to test the build-rustdoc pipeline to check everything works.
@@ -125,7 +125,7 @@ build-rustdoc: | |||
find "$path" -name '*.html' | xargs -I {} -P "$(nproc)" bash -c 'process_file "$@"' _ {} | |||
} | |||
inject_simple_analytics "./crate-docs" | |||
- echo "<meta http-equiv=refresh content=0;url=polkadot_service/index.html>" > ./crate-docs/index.html | |||
- echo "<meta http-equiv=refresh content=0;url=developer_hub/index.html>" > ./crate-docs/index.html |
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.
pipeline references "developer_hub"
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.
that def needs to be updated, @juangirini worth doing a global search / grep of the workspace for developer_hub before merging this if you haven't already
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.
sorry guys, there's been some rebase conflicts that broke the PR after changing its base to master, I'll try to fix this PR, otherwise I'll close it and open a new one
@@ -0,0 +1,14 @@ | |||
flowchart | |||
parity[paritytech.github.io] --> devhub[developer_hub] |
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.
should reference the folder or the crate:
- "docs/sdk"
- "sdk-docs"
@@ -0,0 +1,54 @@ | |||
<script> |
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.
rust docs template references "developer_hub"
//! * [`substrate`], for an overview of what Substrate as the main blockchain framework of Polkadot | ||
//! SDK. | ||
//! * [`frame`], to learn about how to write blockchain applications aka. "App Chains". | ||
//! * Continue with the [`developer_hub`'s "getting started"](crate#getting-started). |
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.
developer_hub
@@ -34,14 +30,21 @@ | |||
//! > **F**ramework for **R**untime **A**ggregation of **M**odularized **E**ntities: Substrate's | |||
//! > State Transition Function (Runtime) Framework. | |||
//! | |||
//! ## Documentation | |||
//! | |||
//! See [`polkadot_sdk::frame`](../developer_hub/polkadot_sdk/frame_runtime/index.html). |
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.
developer_hub
regarding the crate name, do you think it is worth to consider "sdk-docs" vs "polkadot-sdk-docs"? |
closed in favour of #2598 |
This PR is a continuation of #2102 and part of an initiative started here https://hackmd.io/@romanp/rJ318ZCEp What has been done: - The content under `docs/*` (with the exception of `docs/mermaid`) has been moved to `docs/contributor/` - Developer Hub has been renamed to Polkadot SDK Docs, and the crate has been renamed from `developer-hub` to `polkadot-sdk-docs` - The content under `developer-hub/*` has been moved to `docs/sdk` --- Original PR #2565, it has been close due to too many rebase conflicts --------- Co-authored-by: Serban Iorga <[email protected]> Co-authored-by: Chevdor <[email protected]> Co-authored-by: Egor_P <[email protected]> Co-authored-by: Bastian Köcher <[email protected]>
This PR is a continuation of paritytech#2102 and part of an initiative started here https://hackmd.io/@romanp/rJ318ZCEp What has been done: - The content under `docs/*` (with the exception of `docs/mermaid`) has been moved to `docs/contributor/` - Developer Hub has been renamed to Polkadot SDK Docs, and the crate has been renamed from `developer-hub` to `polkadot-sdk-docs` - The content under `developer-hub/*` has been moved to `docs/sdk` --- Original PR paritytech#2565, it has been close due to too many rebase conflicts --------- Co-authored-by: Serban Iorga <[email protected]> Co-authored-by: Chevdor <[email protected]> Co-authored-by: Egor_P <[email protected]> Co-authored-by: Bastian Köcher <[email protected]>
This PR is a continuation of #2102 and part of an initiative started here https://hackmd.io/@romanp/rJ318ZCEp
What has been done:
docs/*
(with the exception ofdocs/mermaid
) has been moved todocs/contributor/
developer-hub
tosdk-docs
developer-hub/*
has been moved todocs/sdk