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

Integrates sbt-org-policies plugin #9

Merged
merged 1 commit into from
Mar 28, 2017

Conversation

juanpedromoreno
Copy link
Contributor

This integration has no effect on the deployed application since we are releasing a snapshot new version.

There are multiples changes across the source code, most of them due to scalafmt autoformatting or file headers. The important changes here are:

  • build.sbt
  • project/plugins.sbt
  • project/build.properties
  • project/ProjectPlugin.scala
  • README.md
  • travis.yml
  • version.sbt

Please, @raulraja could you review? Thanks!

@juanpedromoreno juanpedromoreno merged commit 59fc5ca into master Mar 28, 2017
@juanpedromoreno juanpedromoreno deleted the sbt-org-plugin-integration branch March 28, 2017 18:17
def creationAndManipulation(res0: String): Unit = {
val aliceAccount = new BankAccount
val c3 = Note("C", "Quarter", 3)
val c3 = Note("C", "Quarter", 3)
Copy link
Collaborator

Choose a reason for hiding this comment

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

I’m not sure that aligning definitions like so actually improves readability…

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Agreed, it was just a proposal for code formatting. We just wanted to have a tool to automatize the code formatting.

Please, feel free to modify the code style here: https://github.com/scala-exercises/exercises-scalatutorial/blob/master/.scalafmt.conf

Thanks.

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.

3 participants