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.

Sign up for GitHub

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

Jump to bottom

Move docs dependencies to common location and check for broken docs on pull request #6919

Open
wants to merge 5 commits into
base: master
Choose a base branch
from
Open

Move docs dependencies to common location and check for broken docs on pull request #6919

wants to merge 5 commits into from
+24 −15

Conversation

smokestacklightnin
Copy link
Contributor

This PR accomplishes two main things

  • Move the docs dependencies to docs-requirements.txt. Before, the requirements were repeated in tfx/dependencies.py and in the github docs workflow.
  • Build but do not deploy the docs on pull request. This is to check if any PR will break the docs.

@smokestacklightnin
Move dependencies for docs to requirements-docs.txt so they don't n…
1f36d90 
…eed to be duplicated in the github docs workflow
peytondmurray
peytondmurray requested changes Sep 29, 2024
View reviewed changes
.github/workflows/cd-docs.yml Outdated Show resolved Hide resolved
tfx/dependencies.py Show resolved Hide resolved
@keerthanakadiri keerthanakadiri self-assigned this Sep 30, 2024
peytondmurray
peytondmurray approved these changes Oct 1, 2024
View reviewed changes
Copy link
Contributor

@peytondmurray peytondmurray left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Looks good, except for the comment about verbose mode. I'll approve now, please remove the verbose flag unless it's actually needed here.

@peytondmurray peytondmurray self-requested a review October 1, 2024 18:06
peytondmurray
peytondmurray approved these changes Oct 1, 2024
View reviewed changes
.github/workflows/cd-docs.yml Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@peytondmurray peytondmurray peytondmurray approved these changes

@lego0901 lego0901 Awaiting requested review from lego0901

@roseayeon roseayeon Awaiting requested review from roseayeon

Assignees

@keerthanakadiri keerthanakadiri

Labels
type:docs
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@smokestacklightnin @peytondmurray @keerthanakadiri