Skip to content
This repository was archived by the owner on Nov 7, 2023. It is now read-only.

Conversation

@nurugger07
Copy link

Removes Phoenix from the imported deps in the formatter config and formats the project. Plug should remain only because we are implementing plugs so importing the formatter config makes sense. Although, truth be told I haven't checked the Plug format config so there might not be anything there to be concerned with :)

@nurugger07
Copy link
Author

As noted in a discussion this morning with the team. What does everyone think about just fixing the .formatter.exs file in a PR and then "spot" formatting as we touch files?

Copy link

@tapickell tapickell left a comment

Choose a reason for hiding this comment

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

👍

@moxley
Copy link

moxley commented Mar 21, 2019

As noted in a discussion this morning with the team. What does everyone think about just fixing the .formatter.exs file in a PR and then "spot" formatting as we touch files?

I'm ok either way, but I prefer to merge this PR so we can move on to other things.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants