-
Notifications
You must be signed in to change notification settings - Fork 653
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-#4082: Add pdf
/epub
/htmlzip
formats for doc builds
#4083
Conversation
Codecov Report
@@ Coverage Diff @@
## master #4083 +/- ##
===========================================
- Coverage 85.61% 63.48% -22.14%
===========================================
Files 201 201
Lines 16852 16852
===========================================
- Hits 14428 10698 -3730
- Misses 2424 6154 +3730
Continue to review full report at Codecov.
|
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.
Do we need to specify htmlzip
in addition to epub
and pdf
? We have all three versions available for the 0.11.* and 0.12.* releases. In that case, I'd recommend just changing it to - all
!
Nice catch! |
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.
LGTM! Could you please add this commit + your username to the 0.14.0 release notes, as per #4078? Once you do, I'm happy to approve and merge!
pdf
/epub
formats for doc buildspdf
/epub
/htmlzip
formats for doc builds
…uilds Signed-off-by: Alexey Prutskov <alexey.prutskov@intel.com>
60c4a04
to
e769481
Compare
@@ -23,6 +23,7 @@ Key Features and Updates | |||
* | |||
* Documentation improvements | |||
* DOCS-#4077: Add release notes template to docs folder (#4078) | |||
* DOCS-#4082: Add pdf/epub/htmlzip formats for doc builds (#4083) |
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.
@RehanSD I've added a type/issue number/PR name/ PR number as it's described in example above. But I didn't find an example where I should place my github username. Should it be placed in "Contributors" section or not? Example doesn't show where I should put it
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.
Yes please! For context, this is the release description you see here, so please add your name to the Contributors section with an @
so GitHub can also auto link your profile!
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.
Type/Issue/Pr Title + Number look perfect! I suspect Devin forgot to add his name to the contributors, which I'll fix when I add my docs fix PR that got merged last night to this template!
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.
Looks great to me!
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.
LGTM
Signed-off-by: Alexey Prutskov alexey.prutskov@intel.com
What do these changes do?
PR adds missed after #4073 formats of doc builds
flake8 modin/ asv_bench/benchmarks scripts/doc_checker.py
black --check modin/ asv_bench/benchmarks scripts/doc_checker.py
git commit -s
pdf
andepub
doc builds are missed after adding.readthedocs.yaml
#4082docs/development/architecture.rst
is up-to-date