Skip to content

Keep same surrounding whitespace when re-annotating a file #573

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

Merged
merged 1 commit into from
Sep 6, 2018

Conversation

CyborgMaster
Copy link
Contributor

@CyborgMaster CyborgMaster commented Sep 6, 2018

This fixes #555, fixes #472, and possibly more.

This is currently built on top of #571, because it hits the same lines of code. If for some reason you want to merge this one and not #571, I'll be happy to refactor.

@ctran
Copy link
Owner

ctran commented Sep 6, 2018

@CyborgMaster please rebase and resolve conflicts. Thanks!

@CyborgMaster
Copy link
Contributor Author

On it.

Weird, I thought I structured this such that if you merged the previous one first it would just work... hmm... did something unusual happen with the merge?

@CyborgMaster
Copy link
Contributor Author

rebased!

@ctran ctran merged commit e84e22d into ctran:develop Sep 6, 2018
@ctran
Copy link
Owner

ctran commented Sep 6, 2018

@CyborgMaster Sorry for the extra work. I generally prefer to use "Squash and Merge" option which didn't work out for this scenario.

@CyborgMaster
Copy link
Contributor Author

ah hah, that makes sense. No problem. I'll take care of it as well on the last PR.

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.

Preserve whitespace above model annotations run annotate always add a new space
2 participants