Skip to content
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

Publish artifacts on build_docs CircleCI job to get rendered html files #1321

Merged
merged 1 commit into from
Feb 26, 2021

Conversation

NicolasHug
Copy link
Member

Addresses #1316 (comment)

Docs are currently built but the artifacts aren't published so they generated html files aren't accessible. This PR fixes that.

@NicolasHug NicolasHug changed the title WIP tell CircleCI to publish artifacts to publish docs Publish artifacts on build_docs CircleCI job to get rendered html files Feb 26, 2021
@NicolasHug
Copy link
Member Author

Copy link
Contributor

@vincentqb vincentqb left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@vincentqb vincentqb merged commit 48edb32 into pytorch:master Feb 26, 2021
@vincentqb
Copy link
Contributor

@NicolasHug -- I'm assuming this needs to be cherry-picked in the release branch, is that correct?

@NicolasHug
Copy link
Member Author

I don't think it's needed, as we probably don't need to see the artifacts on the release branch?
(I think this is mostly unrelated to the way the "official" stable docs are pushed)

@mthrok
Copy link
Collaborator

mthrok commented Feb 26, 2021

@NicolasHug Thanks! It's been already super helpful #1322

mthrok pushed a commit to mthrok/audio that referenced this pull request Feb 26, 2021
…ndows platform (pytorch#1321)

* Update tutorial to tell user that TcpStore has been enabled on Windows

* Resolve review comments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants