Skip to content

fix(docs): fix docs deps after resolute work#167

Merged
AloizioMacedo merged 2 commits intocanonical:masterfrom
AloizioMacedo:fix-docs-dependencies-for-resolute-migration
Apr 17, 2026
Merged

fix(docs): fix docs deps after resolute work#167
AloizioMacedo merged 2 commits intocanonical:masterfrom
AloizioMacedo:fix-docs-dependencies-for-resolute-migration

Conversation

@AloizioMacedo
Copy link
Copy Markdown
Contributor

This is the low-effort fix. Since the docs build is a separate process, this should be enough.

In the long term, ideally we want to simply install MAAS as a package when building the documentation, and the granular deps in the docs to be pertaining only to the docs itself.

@AloizioMacedo AloizioMacedo self-assigned this Apr 17, 2026
@AloizioMacedo AloizioMacedo requested a review from a team April 17, 2026 19:41
@AloizioMacedo AloizioMacedo added the documentation Improvements or additions to documentation label Apr 17, 2026
Copy link
Copy Markdown
Contributor

@alexsander-souza alexsander-souza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the django version is going to break things

Comment thread rtd-docs/requirements.txt Outdated
Comment thread rtd-docs/requirements.txt Outdated
@AloizioMacedo
Copy link
Copy Markdown
Contributor Author

the django version is going to break things

I changed things to be in line with what you specified to be extra safe.

But worth noticing that these discrepancies are fine (as long as the docs build): the docs building environment is isolated from the code. We just need the "critical" paths to be consistent in order for the code that gathers the OpenAPI spec to work without hitting issues.

Copy link
Copy Markdown
Contributor

@alexsander-souza alexsander-souza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+1

@AloizioMacedo AloizioMacedo enabled auto-merge April 17, 2026 19:56
@AloizioMacedo AloizioMacedo added this pull request to the merge queue Apr 17, 2026
Merged via the queue into canonical:master with commit c988beb Apr 17, 2026
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants