-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
Configuration file accepts .yaml or .yml #8697
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
Comments
We want to gently push people towards using |
Some kind of mention on that page (v2 config file) would be good. I saw on the config file index page (https://docs.readthedocs.io/en/stable/config-file/index.html), other filenames are mentioned as deprecated. |
Technically, you can even write JSON if you want, but we file all of these under hidden use cases that we don't really want to introduce and potentially confuse readers. We've discussed this a bit recently and want to instead provide a single, clear direction here for these steps. |
Sure. But I was confused as I returned to RtD after a year of not touching the config file. I saw "must ... be named" and I thought maybe my file called differently wasn't even being used. Perhaps tweak the wording like:
|
I think the wording gets too complicated. "Should" invites people to call it... something else? Like Again, I think this is a tradeoff between documenting reality (that we accept several filenames) and keeping documentation simple to better serve users (give just one way to call it, so they don't have to make decisions and they align with what we want from them). |
I understand that we decided to not update the documentation to mention all the variants (it's a regex actually), so I'm going to close this issue in favor of having the documentation clear and simple. |
- Add build configuration. - Rename .yml -> .yaml (See discussion in readthedocs/readthedocs.org#8697)
Details
The config file docs say:
But I've been getting away with a
.yml
suffix. Docs could say it works with either?The text was updated successfully, but these errors were encountered: