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

Fix docs build to use tox, as described #803

Merged
merged 1 commit into from
Feb 2, 2023
Merged

Fix docs build to use tox, as described #803

merged 1 commit into from
Feb 2, 2023

Conversation

mikemckiernan
Copy link
Member

  • Update tox.ini to use the correct dependencies file.
  • Update the docs requirements file to remove stale dependencies.
  • Update the GitHub workflow so that docs are built on a push to the default branch rather than on a schedule.

@mikemckiernan mikemckiernan self-assigned this Jan 27, 2023
@mikemckiernan mikemckiernan added the chore Infrastructure update label Jan 27, 2023
@mikemckiernan
Copy link
Member Author

FYI @radekosmulski , thank you again for letting me know that the docs README did not line up with reality.

@karlhigley karlhigley requested a review from jperez999 January 27, 2023 17:04
@github-actions
Copy link

Documentation preview

https://nvidia-merlin.github.io/Merlin/review/pr-803

@mikemckiernan
Fix docs build to use tox, as described
625850f 
- Update tox.ini to use the correct
  dependencies file.
- Update the docs requirements file
  to remove stale dependencies.
- Update the GitHub workflow so that
  docs are built on a push to the
  default branch rather than on
  a schedule.
@karlhigley karlhigley merged commit b1dac7c into main Feb 2, 2023
@mikemckiernan mikemckiernan deleted the fix-docs-tox branch February 2, 2023 16:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jperez999 jperez999 jperez999 approved these changes

Assignees

@mikemckiernan mikemckiernan

Labels
chore Infrastructure update
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@mikemckiernan @jperez999 @karlhigley