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

Run the formatter on lib/elixir/test/elixir/calendar_test.exs #6866

Merged
merged 1 commit into from
Oct 11, 2017

Conversation

nbw
Copy link
Contributor

@nbw nbw commented Oct 11, 2017

Formatter worked just dandy, but I added a new line before a few pipes then re-ran it.

1) After running the formatter once:

screenshot 2017-10-10 21 35 19

2) After adding a return before the pipe, then rerunning the formatter

screenshot 2017-10-10 21 36 05

I find #2 easier to read, personally, but if you'd prefer #1 then I'll edit and commit it.

@josevalim josevalim merged commit 84f510a into elixir-lang:master Oct 11, 2017
@josevalim
Copy link
Member

❤️ 💚 💙 💛 💜

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

Successfully merging this pull request may close these issues.

None yet

2 participants