Skip to content

Comments

Avert RTD breakage#4449

Merged
hobu merged 2 commits intoPDAL:masterfrom
j9ac9k:avert-rtd-breakage
Jul 17, 2024
Merged

Avert RTD breakage#4449
hobu merged 2 commits intoPDAL:masterfrom
j9ac9k:avert-rtd-breakage

Conversation

@j9ac9k
Copy link
Member

@j9ac9k j9ac9k commented Jul 17, 2024

ReadTheDocs will stop adding extra content and configurations on sphinx builds. RTD automatically sets the html_baseurl based on the config inside RTD, and this will stop working soon.

j9ac9k added 2 commits July 17, 2024 13:18
ReadTheDocs will stop adding extra content and configurations on sphinx
builds. RTD automatically sets the html_baseurl based on the config
inside RTD, and this will stop working soon.

This commit ensures that we grab the baseurl from the environment
variable that RTD sets.
Previous regex for version number would only match to single digit
versions for major.minor.patch (so 2.7.10 would match to 2.7.1).

Also use walrus operator to get rid of extra `None` check.
@hobu hobu merged commit d1b1ecd into PDAL:master Jul 17, 2024
hobu pushed a commit that referenced this pull request Jul 17, 2024
* Avert RTD Upcoming breakage

ReadTheDocs will stop adding extra content and configurations on sphinx
builds. RTD automatically sets the html_baseurl based on the config
inside RTD, and this will stop working soon.

This commit ensures that we grab the baseurl from the environment
variable that RTD sets.

* Regex for version with more than one digit

Previous regex for version number would only match to single digit
versions for major.minor.patch (so 2.7.10 would match to 2.7.1).

Also use walrus operator to get rid of extra `None` check.
@j9ac9k j9ac9k deleted the avert-rtd-breakage branch July 17, 2024 11:53
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.

2 participants