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

docs: add info to support nested workflow, part 1 #313

Merged
merged 2 commits into from
Dec 13, 2024

Conversation

minaelee
Copy link
Contributor

@minaelee minaelee commented Dec 5, 2024

Updates to support the nested workflow where the docs are always in a subfolder (such as "docs/") of a project's root folder. This PR ensures that commands are accompanied by instructions on where to perform them (any location if a system-wide command; within the documentation folder if that is necessary).

Also improved the explanation about integration with the main Makefile of the project; made some minor edits, such as using folder consistently instead of folder and directory inconsistently; added an example of how to configure the documentation folder in RTD.

Note: Other docs pages still require updates to support the nested workflow, but they are blocked by waiting for the init.sh script to be replaced with manual instructions.

@minaelee minaelee requested a review from SecondSkoll December 5, 2024 01:11
@SecondSkoll
Copy link
Collaborator

This looks fine to me, but I'm going to wait for the init removal to be processed as there might be some conflicts and this will be the easier PR to change. There might be some other changes associated with that change that need to be mentioned - though nothing occurs to me right now.

@akcano
Copy link
Contributor

akcano commented Dec 9, 2024

just in case, #314 is merged

@SecondSkoll
Copy link
Collaborator

As I thought, there are lots of conflicts. If you don't have time to resolve these @minaelee I can take your work here and rebase it in a new PR - as it was my decision to delay this merge.

@minaelee
Copy link
Contributor Author

As I thought, there are lots of conflicts. If you don't have time to resolve these @minaelee I can take your work here and rebase it in a new PR - as it was my decision to delay this merge.

No worries, resolved!

@SecondSkoll SecondSkoll merged commit 898410d into canonical:main Dec 13, 2024
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants