-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
Build: implementation of build.commands
#9150
Merged
Merged
Changes from all commits
Commits
Show all changes
37 commits
Select commit
Hold shift + click to select a range
88e942f
Build: POC of `build.commands`
humitos e578554
Docs: reference for `build.commands`
humitos 5365087
Build: solve problem with sanitized output
humitos 2286206
Build: decouple `install_build_tools` from `PythonEnvironment`
humitos 97412b2
Template: add warning note on build details' page
humitos a22e3bd
Build: fail the build if `build.commands` without `output/` folder
humitos 27d1781
Test: `build.commands` test cases
humitos e55d008
Merge branch 'main' of github.com:readthedocs/readthedocs.org into hu…
humitos 243fde5
Typo in order of arguments
humitos 66b41a8
Tests: fix mock patch's path
humitos 935664a
Tests: compare against "" since we are not returning `None`
humitos 958c2ce
Docs: mention that integrations aren't supported
humitos 108357d
Docs: briefly mention to `build.commands` in "Build Customization"
humitos 7f233ea
Use `.. code-block` because it accepts `:caption:`
humitos beeed60
Links with `:ref:` fixed
humitos 464afbb
Remove caption
humitos 2e4579a
Link to the correct page from UI
humitos e2f90b6
Docs: introduce extend and override at the beginning of the doc
humitos 5c3e7cf
Docs: add caption to YAML examples
humitos f30f867
Docs: improves from suggestions
humitos 6adb483
Build: make output path a constant and better error message
humitos c24c691
Build: use `_readthedocs/html` as output folder
humitos 8b1fc27
Merge branch 'main' of github.com:readthedocs/readthedocs.org into hu…
humitos ad4eb4a
Build: import error
humitos 996c97f
Build: return GENERIC doctype when building with `build.commands`
humitos 1a9bf4f
Apply suggestions from code review
humitos d94f339
Build: doctype uses insternal config object
humitos 472750c
Build: detect Python commands that require reshiming
humitos cd6f3c8
Build: `config.build.commands` is always present
humitos 99efb00
Build: update the `Version.documentation_type` properly
humitos 338fde5
Build: check for `commands` and emptyness
humitos b72906e
Test: `build.commands` and `Version.documentation_type` update
humitos 4b7c14f
Build: update `Version.documentation_type` properly
humitos b5d168a
Build: define `reshim_commands` as set
humitos 61bb432
Docs: minor typo
humitos 632eb7c
Build: improve error message when no output folder
humitos 9df29dd
Build: bugfix when working with sets and tuples
humitos File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
31 changes: 31 additions & 0 deletions
31
readthedocs/builds/migrations/0044_alter_version_documentation_type.py
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,31 @@ | ||
# Generated by Django 3.2.13 on 2022-05-30 10:24 | ||
|
||
from django.db import migrations, models | ||
|
||
|
||
class Migration(migrations.Migration): | ||
|
||
dependencies = [ | ||
("builds", "0043_add_cancelled_state"), | ||
] | ||
|
||
operations = [ | ||
migrations.AlterField( | ||
model_name="version", | ||
name="documentation_type", | ||
field=models.CharField( | ||
choices=[ | ||
("sphinx", "Sphinx Html"), | ||
("mkdocs", "Mkdocs"), | ||
("sphinx_htmldir", "Sphinx HtmlDir"), | ||
("sphinx_singlehtml", "Sphinx Single Page HTML"), | ||
("mkdocs_html", "Mkdocs Html Pages"), | ||
("generic", "Generic"), | ||
], | ||
default="sphinx", | ||
help_text="Type of documentation the version was built with.", | ||
max_length=20, | ||
verbose_name="Documentation type", | ||
), | ||
), | ||
] |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This still feels really small compared to how important the information is, but we can improve it over time.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree. This has to be a lot more prominent. However,
I'm thinking we may need a section in the "Build customization" page that list all the considerations required to use
build.commands
or something like that we can link from here. I feel I'm not coming with a good idea of how to write this, tho 🤷🏼