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

Test for v1.2.2 #1426

Merged
merged 8 commits into from
Jan 10, 2024
Merged

Test for v1.2.2 #1426

merged 8 commits into from
Jan 10, 2024

Conversation

mfixstsci
Copy link
Collaborator

BradleySappington and others added 8 commits December 21, 2023 11:50
W503 is line break must be after the binary operator, this has been replaced by W504 which wants line breaks before the binary operator.  I'm tired of getting yelled at for both of them.
Ignore expected claw monitor warnings
Hotfix: Crashing archive_database_update
@pep8speaks
Copy link

Hello @mfixstsci, Thank you for submitting the Pull Request !

Line 156:121: W504 line break after binary operator
Line 157:107: W504 line break after binary operator
Line 366:17: E722 do not use bare 'except'

Line 416:13: E128 continuation line under-indented for visual indent
Line 416:13: W503 line break before binary operator
Line 427:17: E128 continuation line under-indented for visual indent
Line 427:17: W503 line break before binary operator

If you have not done so, please consult the jwql Style Guide

@mfixstsci mfixstsci merged commit d7d6d06 into test Jan 10, 2024
11 checks passed
@mfixstsci mfixstsci changed the title Test for v1.2.3 Test for v1.2.2 Jan 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants