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.

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

How to use a .readthedocs.yaml config file with subprojects? #8234

Closed
qlyoung opened this issue Jun 3, 2021 · 14 comments · Fixed by #10001
Closed

How to use a .readthedocs.yaml config file with subprojects? #8234

qlyoung opened this issue Jun 3, 2021 · 14 comments · Fixed by #10001
Assignees
Labels
Accepted Accepted issue on our roadmap Feature New feature Needed: documentation Documentation is required

Comments

@qlyoung
Copy link

qlyoung commented Jun 3, 2021

I believe this is a question and not a bug, though it could constitute a docs bug.

Details

I'm reading through the documentation on using a config file for RTD available here:
https://docs.readthedocs.io/en/stable/config-file/v2.html#sphinx

My project's docs are composed of a main project and one subproject. The main project corresponds to path doc/user and the subproject corresponds to path doc/developer within the repository; we maintain two separate manuals, one for the user docs and one for development docs. Presently I have this set up properly using the RTD web interface and things work swimmingly. However, I'd like to switch to a config file for various reasons.

In the RTD docs, I don't see how I could express subprojects in the config file.

I apologize if I missed the answer in the docs or in a previous issue.

@stsewd
Copy link
Member

stsewd commented Jun 7, 2021

Hi, currently having several config files in the same repo isn't supported. We were discussing having an env var that users could add to read the config file from that location, so they could use the same repo imported twice.

@qlyoung
Copy link
Author

qlyoung commented Jun 8, 2021

Ok. Should I submit a PR to clarify the docs in this respect? They are very pro-config file but should mention that the config file doesn't support subprojects.

@stsewd
Copy link
Member

stsewd commented Jun 8, 2021

Ok. Should I submit a PR to clarify the docs in this respect? They are very pro-config file but should mention that the config file doesn't support subprojects.

Sure, but I'd use "It doesn't support multiple docs projects in the same repo" or "it doesn't support using multiple config files in the same repo", as subprojects sounds more like git subprojects, which we do support.

@qlyoung
Copy link
Author

qlyoung commented Jun 8, 2021

Ok, I can make this clear. Though I will note there's an entire section titled Subprojects, which ought to provide context.

@humitos
Copy link
Member

humitos commented Jun 9, 2021

Yeah, Read the Docs uses the concept of "Project" and "Sub-project". Git, on the other hand, uses "Sub-module".

@qlyoung
Copy link
Author

qlyoung commented Jun 10, 2021

@stsewd just to make sure - while you say that one repo can't support multiple config files there's no way to specify multiple projects in one config file, right? Conceptually that is what I expected when I have a project that has a subproject.

@stsewd
Copy link
Member

stsewd commented Jun 10, 2021

@stsewd just to make sure - while you say that one repo can't support multiple config files there's no way to specify multiple projects in one config file, right? Conceptually that is what I expected when I have a project that has a subproject.

Yeah that's it, you can import the same project, but can't use a different config file.

@stsewd stsewd added the Needed: documentation Documentation is required label Jul 1, 2021
@KennethNielsen
Copy link

@qlyoung did you ever open the separate issue for this?

@qlyoung
Copy link
Author

qlyoung commented Dec 24, 2021

@KennethNielsen I did not, and I didn't submit a pull request either.

@KennethNielsen
Copy link

Ok. I think I will file it as a feature request then. I don't have the bandwidth to attempt a PR, but it can't hurt to raise it as a feature request.

@stsewd
Copy link
Member

stsewd commented Jan 3, 2022

This feature is in our roadmap, so hopefully will be implemented in the next months.

@humitos humitos added Accepted Accepted issue on our roadmap Feature New feature labels Jan 4, 2022
@ewdurbin
Copy link
Contributor

ewdurbin commented Feb 7, 2023

Is this still planned?

@stsewd
Copy link
Member

stsewd commented Feb 7, 2023

@ewdurbin hi there, supporting multiple config filer per project is still pending a design decision at #8811, the issue here is about documenting the current limitation.

@github-project-automation github-project-automation bot moved this to Planned in 📍Roadmap Feb 28, 2023
@ericholscher ericholscher linked a pull request Mar 15, 2023 that will close this issue
@ericholscher ericholscher moved this from Planned to In progress in 📍Roadmap Mar 15, 2023
@github-project-automation github-project-automation bot moved this from In progress to Done in 📍Roadmap Apr 17, 2023
@benjaoming
Copy link
Contributor

A quick update on why this issue is now closed and fixed:

The proposal from @ewdurbin in #10001 was adopted and will premiere in the next deployment. If there's any feedback, please don't hesitate to write here or open a new issue 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accepted Accepted issue on our roadmap Feature New feature Needed: documentation Documentation is required
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

6 participants