Skip to content

Conversation

@yhy808
Copy link
Collaborator

@yhy808 yhy808 commented Dec 16, 2024

No description provided.

@pep8speaks
Copy link

pep8speaks commented Dec 16, 2024

Hello @yhy808! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 590:9: E722 do not use bare 'except'
Line 603:9: E722 do not use bare 'except'
Line 608:80: E501 line too long (86 > 79 characters)
Line 611:9: E722 do not use bare 'except'
Line 615:80: E501 line too long (88 > 79 characters)
Line 619:9: E722 do not use bare 'except'
Line 630:80: E501 line too long (84 > 79 characters)
Line 668:80: E501 line too long (99 > 79 characters)
Line 761:80: E501 line too long (100 > 79 characters)
Line 765:5: E722 do not use bare 'except'
Line 772:80: E501 line too long (83 > 79 characters)
Line 782:80: E501 line too long (80 > 79 characters)
Line 788:80: E501 line too long (81 > 79 characters)
Line 798:80: E501 line too long (99 > 79 characters)
Line 868:80: E501 line too long (85 > 79 characters)
Line 875:80: E501 line too long (81 > 79 characters)
Line 1138:34: E203 whitespace before ':'
Line 1141:1: E402 module level import not at top of file
Line 1145:80: E501 line too long (95 > 79 characters)
Line 1148:80: E501 line too long (80 > 79 characters)
Line 1153:80: E501 line too long (102 > 79 characters)
Line 1161:1: E402 module level import not at top of file
Line 1176:80: E501 line too long (87 > 79 characters)
Line 1202:1: E402 module level import not at top of file
Line 1203:1: E402 module level import not at top of file
Line 1216:80: E501 line too long (82 > 79 characters)
Line 1245:80: E501 line too long (81 > 79 characters)
Line 1246:80: E501 line too long (82 > 79 characters)
Line 1248:80: E501 line too long (83 > 79 characters)
Line 1254:80: E501 line too long (103 > 79 characters)
Line 1275:80: E501 line too long (80 > 79 characters)
Line 1278:80: E501 line too long (107 > 79 characters)
Line 1288:80: E501 line too long (83 > 79 characters)
Line 1299:80: E501 line too long (90 > 79 characters)
Line 1320:80: E501 line too long (93 > 79 characters)
Line 1331:13: E115 expected an indented block (comment)
Line 1338:80: E501 line too long (89 > 79 characters)
Line 1349:9: E115 expected an indented block (comment)
Line 1351:1: W293 blank line contains whitespace

Comment last updated at 2025-01-17 23:30:09 UTC

for field in ["title", "journal", "authors", "year"]
)

crossref_text = " ".join(
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

trivial comment: crossref_text is the same thing every pass of the loop, so it should be defined before the loop

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