Skip to content

Conversation

@techtonik
Copy link
Contributor

@techtonik techtonik commented Sep 4, 2019

Fixes #112

@techtonik techtonik mentioned this pull request Sep 4, 2019
Remove duplicate code from is_different_lines().
Rename new_lines to new_text where it is not list
Copy link
Owner

@peterbe peterbe left a comment

Choose a reason for hiding this comment

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

It needs a unit test that reflects what you mentioned in the issue. Great work!!

@peterbe peterbe changed the title Support indented requirements (fixes #112) Support indented requirements Sep 4, 2019
@peterbe
Copy link
Owner

peterbe commented Sep 4, 2019

Note that I edited the PR title and PR description so GitHub would know to automatically close the issue when this PR lands.

Copy link
Owner

@peterbe peterbe left a comment

Choose a reason for hiding this comment

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

Wonderful!

@peterbe peterbe merged commit b42eeb1 into peterbe:master Sep 5, 2019
@techtonik
Copy link
Contributor Author

Added the test. Debugging missing trailing \n in passed new_lines[2] was far from pleasant.

@techtonik techtonik deleted the support-indent branch September 5, 2019 12:19
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.

Handle indentation

2 participants