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

CI: publish rustdocs #9527

Merged
merged 14 commits into from
Aug 12, 2021
Merged

CI: publish rustdocs #9527

merged 14 commits into from
Aug 12, 2021

Conversation

TriplEight
Copy link
Contributor

@TriplEight TriplEight commented Aug 9, 2021

Publish docs from master to https://paritytech.github.io/substrate/

  • remove publish-s3-doc: as it seems that no one actually used them
  • build-rust-doc: is not allowed to fail

@TriplEight TriplEight added A3-in_progress Pull request is in progress. No review needed at this stage. B0-silent Changes should not be mentioned in any release notes C3-medium PR touches the given topic and has a medium impact on builders. labels Aug 9, 2021
@TriplEight TriplEight self-assigned this Aug 9, 2021
@TriplEight TriplEight requested a review from a team as a code owner August 9, 2021 15:58
@TriplEight TriplEight added A0-please_review Pull request needs code review. and removed A3-in_progress Pull request is in progress. No review needed at this stage. labels Aug 9, 2021
@TriplEight TriplEight requested review from s3krit and bkchr August 9, 2021 19:06
Copy link
Contributor

@sergejparity sergejparity left a comment

Choose a reason for hiding this comment

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

Looks good to me

@TriplEight
Copy link
Contributor Author

It might be that it's not needed, I'm currently trying to guess how https://substrate.dev/rustdocs/ gets updated.

@TriplEight TriplEight merged commit ee76b9f into master Aug 12, 2021
@TriplEight TriplEight deleted the 3x8_publish_rustdocs branch August 12, 2021 08:13
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 C3-medium PR touches the given topic and has a medium impact on builders.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants