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 add-handler putting two routes on one line #922

Merged
merged 1 commit into from Feb 4, 2015

Conversation

Projects
None yet
2 participants
@MaxGabriel
Member

MaxGabriel commented Feb 4, 2015

  • If there is a new line at the end of the file, add the route as normal
  • If there isn't, add a newline character before the route
  • Closes #921

I tested this on OS X. I'm not super familiar with what type of newlines Windows programmers use; I'm not sure if that's an issue here.

Fix add-handler putting two routes on one line
* If there is a new line at the end of the file, add the route as normal
* If there isn't, add a newline character before the route
* Closes #921

snoyberg added a commit that referenced this pull request Feb 4, 2015

Merge pull request #922 from MaxGabriel/fixAddHandlerNewlines
Fix add-handler putting two routes on one line

@snoyberg snoyberg merged commit f3fea0a into yesodweb:master Feb 4, 2015

1 check was pending

continuous-integration/travis-ci The Travis CI build is in progress
Details
@snoyberg

This comment has been minimized.

Show comment
Hide comment
@snoyberg

snoyberg Feb 4, 2015

Member

Thanks!

Member

snoyberg commented Feb 4, 2015

Thanks!

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