Conversation
|
FWIW, forked repos will be supported once I finish drdoctr/doctr#343, assuming that you configure doctr for that repo. The main issue is if you configure a base repo then a fork cannot work, as Travis doesn't give forks or forked pull requests access to the secure environment variables for obvious reasons. |
mrakitin
left a comment
There was a problem hiding this comment.
I think this warning can be better represented via a warning block.
I also think the link from @asmeurer's #79 (comment) can be mentioned as well.
Co-Authored-By: Maksim Rakitin <mrakitin@users.noreply.github.com>
mrakitin
left a comment
There was a problem hiding this comment.
Looks good. Here are a couple of my suggestions.
Co-Authored-By: Maksim Rakitin <mrakitin@users.noreply.github.com>
Co-Authored-By: Maksim Rakitin <mrakitin@users.noreply.github.com>
|
Good to know. At least some directives will be confused by the lack of a blank line, as they will interpret the line beneath |
|
Sounds good, will do in the future. |
DOC: update doctr instruction

Reason: https://github.com/drdoctr/doctr/blob/0f19ff78c8239efcc98d417f36b0a31d9be01ba5/doctr/travis.py#L593
It's very hard to find in Travis log. Think need to mention in instruction.