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
Docstring formatting and linting using pydocstyle
and darglint
#2670
Merged
Merged
Changes from 33 commits
Commits
Show all changes
81 commits
Select commit
Hold shift + click to select a range
69e7b11
init
angela97lin 82f358c
add auto
angela97lin 61f2207
merge
angela97lin fa838da
Merge branch 'main' into 878_docstring_formatting
angela97lin 542eb74
do some more cleanup
angela97lin f74c174
Merge branch '878_docstring_formatting' of github.com:alteryx/evalml …
angela97lin 28e67a2
fix some data checks
angela97lin 3752f8b
start to add lines
angela97lin 4ba3d25
dashes under sections
angela97lin 0cec01d
sphinx dont tree warnings as errors, need to remove later but using t…
angela97lin 0801840
try update make bat
angela97lin 8c646c8
try to fix class imbal dc as example
angela97lin c6a0e54
empty commit
angela97lin 2bdbb99
change to parameters
angela97lin 677385c
fix data checks
angela97lin 4674ea6
cleanup more
angela97lin 3ad3847
fixing more
angela97lin cf45bb7
add ignore test directory flag to lint command
angela97lin 7bc5d94
more cleanup
angela97lin 6849d3a
merge main
angela97lin 7049183
adding more fixes
angela97lin 3a06632
Merge branch 'main' into 878_docstring_formatting
angela97lin aee0e45
revert tests and clean more
angela97lin 8afc300
Merge branch '878_docstring_formatting' of github.com:alteryx/evalml …
angela97lin 913dc99
merge
angela97lin 70572a0
clean up objectives
angela97lin f7bfb53
release notes update
angela97lin ba685e3
fix tests
angela97lin 4d302d0
fix dependencies
angela97lin c452033
cleanup
angela97lin 0b37cfa
more cleanup
angela97lin 29bd507
yet more cleanup
angela97lin 43f55c3
a final ounce of cleanup
angela97lin 5e0da3f
try darglint
angela97lin 1aa06ad
merge master
angela97lin c36071c
some fixing linting
angela97lin b980a8e
more fixing
angela97lin f8255f7
some more cleanup
angela97lin f3491f9
more cleanup
angela97lin f8652ba
merge main
angela97lin f41f9e8
merge main
angela97lin 5b15cd6
more cleanup
angela97lin fd43934
clean up
angela97lin b956831
mid merge, need to address new comments
angela97lin 2c0c74a
linting
angela97lin e4bd693
try rtd
angela97lin b58e3ed
try
angela97lin 76b7879
try without bullets
angela97lin 07a4f93
revert graphs.py to see what changes
angela97lin 5f48e12
add back a little
angela97lin a5d4b12
add back a little more
angela97lin f0fd9aa
Merge branch 'main' into 878_docstring_formatting
angela97lin 1111632
add back more, ignoring indentation errors likely causing RtD failures
angela97lin bea2dcb
Merge branch '878_docstring_formatting' of github.com:alteryx/evalml …
angela97lin 970f246
test kwargs
angela97lin 0c67410
add another kwargs
angela97lin 10ab286
attempt to fix kwargs
angela97lin 622342b
lost attempt again at fixing kwargs
angela97lin eb94c89
remove to confirm issue
angela97lin 62bfa26
revert
angela97lin 59114c9
Merge branch 'main' into 878_docstring_formatting
angela97lin 067595d
revert
angela97lin f66fcc5
???
angela97lin 11cd30e
change indentation
angela97lin 3c889a8
try
angela97lin 6720618
try again
angela97lin 39ca658
test square brackets
angela97lin 9b4e2cc
test underscore
angela97lin 2a166d8
remove raises
angela97lin 9a98ba8
backslash
angela97lin 32d5cc1
try removal
angela97lin 4fc6db7
test
angela97lin eea28fc
clean up setup
angela97lin d45408c
Merge branch 'main' into 878_docstring_formatting
angela97lin 57941fc
cleanup and revert accidental merge diffs
angela97lin 4fbaa63
Merge branch '878_docstring_formatting' of github.com:alteryx/evalml …
angela97lin da14413
Merge branch 'main' into 878_docstring_formatting
chukarsten a818536
clean up from comments
angela97lin 1da0d4d
Merge branch 'main' into 878_docstring_formatting
angela97lin f0bc3db
add default to contributing
angela97lin 8f714bf
Merge branch '878_docstring_formatting' of github.com:alteryx/evalml …
angela97lin File filter
Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
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
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -4,4 +4,4 @@ | |
flake8==3.7.0 | ||
black==21.5b1 | ||
isort==5.0.0 | ||
|
||
pydocstyle==6.1.1 |
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
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
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
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 |
---|---|---|
@@ -1,3 +1,4 @@ | ||
"""AutoML algorithms that power EvalML.""" | ||
from .automl_algorithm import AutoMLAlgorithm, AutoMLAlgorithmException | ||
from .iterative_algorithm import IterativeAlgorithm | ||
from .default_algorithm import DefaultAlgorithm |
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.
Reads: use Google convention (since that was mostly what our convention was) but add D400 (add period to first line). Match all directories that don't start with
test
.