Skip to content

Fix readthedocs build fail - #35939

Merged
irtazaakram merged 2 commits into
masterfrom
fix-readthedocs
Dec 6, 2024
Merged

Fix readthedocs build fail#35939
irtazaakram merged 2 commits into
masterfrom
fix-readthedocs

Conversation

@irtazaakram

Copy link
Copy Markdown
Member

@feanil feanil left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks like this helps build locally, but is still taking a long time on readthedocs. Perhaps we should disable all the autoapi_dirs for now to unblock other docs.

@irtazaakram
irtazaakram requested a review from a team as a code owner December 5, 2024 06:52
@irtazaakram

Copy link
Copy Markdown
Member Author

I've disabled autoapi here be0f5cf.

@irtazaakram

Copy link
Copy Markdown
Member Author

@feanil feanil left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Looks good, can you

  • Backport this to the Sumac branch
  • Write up a ticket to investigate this further that we can pick up later.

@irtazaakram

Copy link
Copy Markdown
Member Author

I've created issue here #35977.

@irtazaakram

Copy link
Copy Markdown
Member Author

sumac backport PR #35978

@feanil

feanil commented Dec 6, 2024

Copy link
Copy Markdown
Contributor

Looks great, feel free to merge this and the other PR whenever you're ready.

@irtazaakram
irtazaakram merged commit 0dd9f8f into master Dec 6, 2024
@irtazaakram
irtazaakram deleted the fix-readthedocs branch December 6, 2024 13:58
@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production.

@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

1 similar comment
@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

2U Release Notice: This PR has been rolled back from the edX production environment.

1 similar comment
@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

2U Release Notice: This PR has been rolled back from the edX production environment.

@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

2U Release Notice: This PR has been deployed to the edX staging environment in preparation for a release to production.

@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

1 similar comment
@edx-pipeline-bot

Copy link
Copy Markdown
Contributor

2U Release Notice: This PR has been deployed to the edX production environment.

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