-
Couldn't load subscription status.
- Fork 31
feat: Stage 2 of nox implementation - adding docs target
#473
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
Conversation
|
Docs session test log: |
|
The I'd suggest to remove or at least disable this parallelization for now, as it's benefit/trouble ratio seems to be less than unity. |
As @IlyaFaer points out in #471 (comment), we probably can't outright remove the parallelization without kokoro timing out. But hopefully we can split it into separate parallel kokoro checks. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
A few comments, the only substantial one is about the ../README.md line.
As suggested in #466 , this represents the second item of the recommended list aimed to implement
noxtesting automation in multiple steps.Change list:
docstarget added to Nox configuration;readmeandchangelogfiles;.gitignore.Towards #474.