Skip to content

Conversation

@harshil21
Copy link
Member

@harshil21 harshil21 commented Jan 2, 2023

Uploads the built docs from the workflow run so it is easy to see the rendered html pages without building the docs again locally.

Docs on artifacts: https://docs.github.com/en/actions/using-workflows/storing-workflow-data-as-artifacts

How to view the artifact? Click on the "Details" of Test Doc Build -> Summary (left panel) -> scroll down and download the artifact

@harshil21 harshil21 added the ⚙️ documentation affected functionality: documentation label Jan 2, 2023
@harshil21 harshil21 changed the base branch from master to doc-fixes January 2, 2023 20:40
@harshil21 harshil21 changed the base branch from doc-fixes to master January 2, 2023 20:42
@harshil21 harshil21 added this to the v20.1 milestone Jan 2, 2023
Copy link
Member

@Bibo-Joshi Bibo-Joshi left a comment

Choose a reason for hiding this comment

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

Also a nice idea! TBH I guess the use case is not overly big, but neither is the effort 🙃

Copy link
Member

@Bibo-Joshi Bibo-Joshi left a comment

Choose a reason for hiding this comment

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

Thanks for the updates! As said above, even if the advantage may not be huge, I see no downsides of this change. I'll give the others a chance to chime in if they see the need and merge on thursday evening otherwise :)

@Bibo-Joshi Bibo-Joshi merged commit f9cbddd into master Jan 6, 2023
@Bibo-Joshi Bibo-Joshi deleted the doc-artifacts branch January 6, 2023 07:14
@github-actions github-actions bot locked and limited conversation to collaborators Jan 13, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

⚙️ documentation affected functionality: documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants