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

Core structure creation #1

Merged

lunamorrow merged 2 commits into MDAnalysis:main from lunamorrow:main

May 27, 2024

Collaborator

lunamorrow commented May 27, 2024

Starting to build core structure of this converter.

lunamorrow added 2 commits

May 27, 2024 23:14


Add base structure for OpenBabel and OpenBabelParser files

40646b7


quick tweaks to import statements

6016f68

pep8speaks commented May 27, 2024

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

In the file mda_openbabel_converter/OpenBabel.py:

Line 9:1: E302 expected 2 blank lines, found 1
Line 15:76: W291 trailing whitespace
Line 26:1: E302 expected 2 blank lines, found 1
Line 44:25: W292 no newline at end of file
Line 44:25: W292 no newline at end of file
Line 44:25: W292 no newline at end of file

In the file mda_openbabel_converter/OpenBabelParser.py:

Line 8:1: E302 expected 2 blank lines, found 1
Line 10:74: W291 trailing whitespace
Line 11:80: W291 trailing whitespace
Line 16:77: W291 trailing whitespace
Line 17:79: W291 trailing whitespace
Line 27:44: W291 trailing whitespace
Line 29:13: W292 no newline at end of file

lunamorrow merged commit 8f4483c into MDAnalysis:main

1 check passed

lunamorrow self-assigned this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment