Skip to content

Conversation

@samuelgarcia
Copy link
Contributor

No description provided.

@pep8speaks
Copy link

pep8speaks commented Dec 17, 2020

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

Line 42:100: E501 line too long (102 > 99 characters)
Line 63:57: E226 missing whitespace around arithmetic operator

Comment last updated at 2021-03-16 13:06:37 UTC

@apdavison apdavison modified the milestones: 0.10.0, 0.11.0 May 10, 2021
@JuliaSprenger
Copy link
Member

@samuelgarcia Is this PR ready for review or should it be still in draft mode?

@samuelgarcia
Copy link
Contributor Author

Still a draft.
I will close this for now and come on this later.
Maybe triudesclous will change a bit in following month so it is not so releveant any more.

@samuelgarcia samuelgarcia deleted the tridesclous_reader branch March 9, 2022 11:44
@apdavison apdavison modified the milestones: 0.10.3, 0.11.0 Aug 30, 2022
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.

4 participants