Skip to content

PR04 errors fix #25157

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.

Sign up for GitHub

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

Merged
jreback merged 17 commits into from
Feb 11, 2019
Merged

PR04 errors fix #25157

Changes from 1 commit
Commits
Show all changes
17 commits
Select commit Hold shift + click to select a range
0dc07f2
Partial PR04 errors fix
mwoss Feb 5, 2019
b93e438
Fix rest of the R04 errors
mwoss Feb 6, 2019
ae369ec
PEP8 - line too long fix
mwoss Feb 6, 2019
3ac2383
Add additional space between param name and type, add new lines to
mwoss Feb 6, 2019
7651a63
Add PR04 errors check to code_check script
mwoss Feb 6, 2019
93654b9
Merge branch 'master' into issue-25105
mwoss Feb 6, 2019
ce46f24
Additional docs fixes related to PR04 errors
mwoss Feb 6, 2019
e9fdd08
Merge remote-tracking branch 'origin/issue-25105' into issue-25105
mwoss Feb 6, 2019
63aaaf0
Remove unnecessary colons and note section in Grouper docs
mwoss Feb 7, 2019
0447294
Merge branch 'master' into issue-25105
mwoss Feb 7, 2019
aeed6ab
Change parameter order
mwoss Feb 8, 2019
9308491
Merge remote-tracking branch 'origin/issue-25105' into issue-25105
mwoss Feb 8, 2019
4bb3b0b
Remove encoding param in opening named temp file
mwoss Feb 8, 2019
f95c4b5
Merge branch 'master' into issue-25105
mwoss Feb 8, 2019
619253c
Separate types and descriptions
mwoss Feb 10, 2019
8113c21
Merge remote-tracking branch 'origin/issue-25105' into issue-25105
mwoss Feb 10, 2019
bffd111
Remove errant colon
WillAyd Feb 11, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Merge branch 'master' into issue-25105
  • Loading branch information
@mwoss
mwoss authored Feb 7, 2019
commit 0447294358fcda8143cfdf18433974b15528ae9b
4 changes: 2 additions & 2 deletions ci/code_checks.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -240,8 +240,8 @@ fi
### DOCSTRINGS ###
if [[ -z "$CHECK" || "$CHECK" == "docstrings" ]]; then

MSG='Validate docstrings (GL06, GL07, GL09, SS04, PR03, PR04, PR05, EX04, RT04, SS05)' ; echo $MSG
$BASE_DIR/scripts/validate_docstrings.py --format=azure --errors=GL06,GL07,GL09,SS04,PR03,PR04,PR05,EX04,RT04,SS05
MSG='Validate docstrings (GL06, GL07, GL09, SS04, PR03, PR05, EX04, RT04, SS05, SA05)' ; echo $MSG
$BASE_DIR/scripts/validate_docstrings.py --format=azure --errors=GL06,GL07,GL09,SS04,PR03,PR04,PR05,EX04,RT04,SS05,SA05
RET=$(($RET + $?)) ; echo $MSG "DONE"

fi
Expand Down
You are viewing a condensed version of this merge commit. You can view the full changes here.