-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
Docs: Add some messages flagging the upcoming requirement of a .readthedocs.yaml #10389
Conversation
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.
These changes look good other than the tutorial, I think we want to prioritize an update there, not a warning that is a "dead end" for the user.
Co-authored-by: Eric Holscher <[email protected]>
…s/readthedocs.org into docs/deprecate-no-config-file
…cs/deprecate-no-config-file
Customizing the build process | ||
----------------------------- | ||
Adding a configuration file | ||
--------------------------- |
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.
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.
Looks good with a small edit.
Co-authored-by: Eric Holscher <[email protected]>
These are just helpful pointers, documentation should be improved afterwards
Fixes #10372
📚 Documentation previews 📚
docs
): https://docs--10389.org.readthedocs.build/en/10389/dev
): https://dev--10389.org.readthedocs.build/en/10389/