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

Fix #204:decoder.loads:detection for triple quote ending not correct on non stripped line #293

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

rudy-6-4
Copy link

@rudy-6-4 rudy-6-4 commented May 5, 2020

triple quote """ or ''' ending a multiline string are not checked
correctly if line ends with comments or spacing or anything

@rudy-6-4 rudy-6-4 changed the title Fix #204 triple quote detection incorrect on non stipped line Fix #204 triple quote detection incorrect on non stripped line May 6, 2020
@rudy-6-4 rudy-6-4 changed the title Fix #204 triple quote detection incorrect on non stripped line Fix #204:decoder.loads:detection for triple quote ending not correct on non stripped line May 6, 2020
@rudy-6-4
Copy link
Author

rudy-6-4 commented May 7, 2020

A better fix is coming with tests also.

triple quote """ or ''' ending a multiline string are not checked
correctly if line ends with comments or spacing or anything
@rudy-6-4 rudy-6-4 force-pushed the fix_parsing_triple_quote branch from 1dcf56b to 87c8c18 Compare May 8, 2020 14:15
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.

1 participant