Skip to content

ci: Add workflow to publish docs#90

Merged
chtruong814 merged 4 commits intomainfrom
chtruong/publish-docs
Jan 30, 2026
Merged

ci: Add workflow to publish docs#90
chtruong814 merged 4 commits intomainfrom
chtruong/publish-docs

Conversation

@chtruong814
Copy link
Collaborator

Add workflow to publish docs

Signed-off-by: Charlie Truong <chtruong@nvidia.com>
emails-csv: ${{ inputs.notify-emails && format('{0},{1}', vars.docs_release_emails, inputs.notify-emails) || vars.docs_release_emails }}
overwrite-latest-on-tag: false
run-on-version-tag-only: ${{ github.ref_name != 'main' }}
request-name: megatron-bridge-publish-docs-${{ github.run_id }}
Copy link
Contributor

Choose a reason for hiding this comment

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

This line needs an update - not sure about the value though

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

oof.

Signed-off-by: Charlie Truong <chtruong@nvidia.com>
@copy-pr-bot
Copy link

copy-pr-bot bot commented Jan 26, 2026

Auto-sync is disabled for draft pull requests in this repository. Workflows must be run manually.

Contributors can view more details about this message here.

@ko3n1g
Copy link
Contributor

ko3n1g commented Jan 26, 2026

Drafting this as we dont have the docs yet

Signed-off-by: Charlie Truong <chtruong@nvidia.com>
@chtruong814
Copy link
Collaborator Author

@ko3n1g what's the harm in merging?

@chtruong814 chtruong814 marked this pull request as ready for review January 29, 2026 21:31
@ko3n1g
Copy link
Contributor

ko3n1g commented Jan 29, 2026

Drafting this as we dont have the docs yet

It doesn't feel great to have an automation enabled before the first official docs publish. Shall we at least deactivate the workflow to prevent accidental runs?

@chtruong814
Copy link
Collaborator Author

@ko3n1g Yeah, that makes sense. I'll disable after merging.

@chtruong814 chtruong814 merged commit 9d83cb5 into main Jan 30, 2026
19 checks passed
@chtruong814 chtruong814 deleted the chtruong/publish-docs branch January 30, 2026 16:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments