-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Migration to pyproject.toml
#3338
base: master
Are you sure you want to change the base?
Changes from 21 commits
e4cddf2
b575804
0c7f24d
87de330
aee8e9c
06df545
5ad74bc
0816898
2e88012
a62ef47
d51c3b0
1ce319b
48b9029
e486da0
c9e2a8f
3e0720d
f9819be
483aa25
62f195a
cb3d1c1
aa22b4e
5e68ba0
d60b65f
77a223d
9bf7c67
68922bf
0e8c7cd
0c94ebb
57b56ea
4428baf
2821873
62fd597
43c2856
844d94d
0038252
1e7b2dc
9616953
12a9d8a
7be8a6d
cd6fee3
48650b7
6f8a76f
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -172,3 +172,5 @@ Icon | |
Network Trash Folder | ||
Temporary Items | ||
.apdisk | ||
|
||
.python-version |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -16,3 +16,5 @@ python: | |
install: | ||
- method: pip | ||
path: . | ||
extra_requirements: | ||
- readthedocs |
This file was deleted.
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -67,7 +67,10 @@ def _warn_node(self, msg, node, *args, **kwargs): | |
|
||
sphinx.environment.BuildEnvironment.warn_node = _warn_node | ||
|
||
if os.environ.get('READTHEDOCS', None) == 'True': | ||
# on_rtd is whether we are on readthedocs.org, this line of code grabbed from docs.readthedocs.org | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Before merging, we'll need to identify how to verify the merging of the final version of this branch doesn't break Luigi's readthedocs. There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. @hirosassa , how you thought how we can be sure that we don't break RTD when merging this PR? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I think it is a little bit difficult because it depends on webhook of this repo. |
||
on_rtd = os.environ.get('READTHEDOCS', None) == 'True' | ||
|
||
if on_rtd: | ||
# Run sphinx-apidoc automatically in readthedocs | ||
# Taken from this: https://lists.torproject.org/pipermail/tor-commits/2012-September/046695.html | ||
os.system('sphinx-apidoc -o api -T ../luigi --separate') | ||
|
@@ -83,7 +86,7 @@ def _warn_node(self, msg, node, *args, **kwargs): | |
# -- General configuration ------------------------------------------------ | ||
|
||
# If your documentation needs a minimal Sphinx version, state it here. | ||
needs_sphinx = '1.4.4' # Value mirrored in doc/conf.py | ||
needs_sphinx = '1.4.4' # Value mirrored in pyproject.toml | ||
|
||
# Add any Sphinx extension module names here, as strings. They can be | ||
# extensions coming with Sphinx (named 'sphinx.ext.*') or your custom | ||
|
@@ -167,9 +170,6 @@ def _warn_node(self, msg, node, *args, **kwargs): | |
# The theme to use for HTML and HTML Help pages. See the documentation for | ||
# a list of builtin themes. | ||
|
||
# on_rtd is whether we are on readthedocs.org, this line of code grabbed from docs.readthedocs.org | ||
on_rtd = os.environ.get('READTHEDOCS', None) == 'True' | ||
|
||
if not on_rtd: # only import and set the theme if we're building docs locally | ||
try: | ||
import sphinx_rtd_theme | ||
|
This file was deleted.
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,3 @@ | ||
# coding: utf-8 | ||
|
||
VERSION = '3.6.0' |
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.
The step just below this line includes a conditional to not run coverage upload if flake8 or docs. But this "others" matrix only includes flake8 and docs. Could we then remove its
Upload coverage to Codecov
step entirely?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.
fixed in 48650b7