Skip to content
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

V1.2.3 [master] #526

Merged
merged 113 commits into from
Feb 2, 2022
Merged

V1.2.3 [master] #526

merged 113 commits into from
Feb 2, 2022

Conversation

bourque
Copy link
Contributor

@bourque bourque commented Jan 19, 2022

This PR contains the v1.2.3 release (into to master branch)

mfixstsci and others added 30 commits June 9, 2021 17:03
Scheduled monthly dependency update for August
@pep8speaks
Copy link

pep8speaks commented Jan 19, 2022

Hello @bourque, Thank you for updating !

Line 65:20: W292 no newline at end of file

Line 142:26: E712 comparison to False should be 'if cond is False:' or 'if not cond:'

Line 667:21: W605 invalid escape sequence '('
Line 667:25: W605 invalid escape sequence ')'
Line 667:57: W605 invalid escape sequence '\c'
Line 667:74: W605 invalid escape sequence '\e'
Line 673:46: W605 invalid escape sequence '('
Line 673:48: W605 invalid escape sequence '\l'
Line 673:56: W605 invalid escape sequence '\m'
Line 673:66: W605 invalid escape sequence '\h'
Line 673:79: W605 invalid escape sequence '\m'
Line 673:85: W605 invalid escape sequence ')'
Line 674:46: W605 invalid escape sequence '('
Line 674:48: W605 invalid escape sequence '\l'
Line 674:56: W605 invalid escape sequence '\m'
Line 674:66: W605 invalid escape sequence '\h'
Line 674:79: W605 invalid escape sequence '\m'
Line 674:85: W605 invalid escape sequence ')'
Line 675:46: W605 invalid escape sequence '('
Line 675:48: W605 invalid escape sequence '\l'
Line 675:56: W605 invalid escape sequence '\m'
Line 675:66: W605 invalid escape sequence '\h'
Line 675:79: W605 invalid escape sequence '\m'
Line 675:85: W605 invalid escape sequence ')'
Line 681:58: W605 invalid escape sequence '('
Line 681:62: W605 invalid escape sequence '\m'
Line 681:69: W605 invalid escape sequence '\m'
Line 681:75: W605 invalid escape sequence ')'

Line 103:46: W605 invalid escape sequence '\m'
Line 442:30: W605 invalid escape sequence '\m'
Line 443:32: W605 invalid escape sequence '\m'
Line 443:39: W605 invalid escape sequence '\m'

Line 62:50: E226 missing whitespace around arithmetic operator
Line 63:50: E226 missing whitespace around arithmetic operator
Line 64:48: E226 missing whitespace around arithmetic operator
Line 64:61: E226 missing whitespace around arithmetic operator
Line 65:47: E226 missing whitespace around arithmetic operator
Line 65:60: E226 missing whitespace around arithmetic operator
Line 72:26: E226 missing whitespace around arithmetic operator
Line 73:18: E226 missing whitespace around arithmetic operator
Line 73:31: E226 missing whitespace around arithmetic operator

Line 65:20: W292 no newline at end of file

Comment last updated at 2022-02-01 20:21:28 UTC

@bourque bourque requested a review from nespinoza January 19, 2022 14:28
@bourque bourque merged commit 0b213ab into master Feb 2, 2022
@bourque bourque deleted the v1.2.3 branch February 2, 2022 19:06
hover2pi pushed a commit that referenced this pull request Jul 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants