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

Switch to using build.sbt, revel in auto-plugin happiness #1358

Merged
merged 1 commit into from
Oct 26, 2016

Conversation

rtyley
Copy link
Member

@rtyley rtyley commented Oct 26, 2016

Using .sbt build definitions (rather than .scala definitions) has been the recommended way of defining SBT projects since SBT v0.13.12, for all project types, including multi-project builds.

Using auto-plugins is much nicer with .sbt build definitions - the necessary imports are all automatically made for you. It's become fiddlier to do .scala-based build definitions because plugin authors no longer document what those necessary imports are!

I've deliberately tried to not shift too many files round with this commit, but at some point it would be nice to bring the sole frontend subproject up to the root level.

http://notes.implicit.ly/post/147554513094/sbt-01312
http://www.scala-sbt.org/0.13/docs/Basic-Def.html
https://groups.google.com/d/msg/sbt-dev/S-pr2nZQ3KY/iX9q3cbrHVQJ

cc @paulbrown1982 @AWare

Using .sbt build definitions (rather than .scala definitions) has been the
recommended way of defining SBT projects since SBT v0.13.12, for all
project types, including multi-project builds.

Using auto-plugins is much nicer with .sbt build definitions - the
necessary imports are all automatically made for you. It's become fiddlier
to do .scala-based build definitions because plugin authors no
longer document what those necessary imports are!

I've deliberately tried to not shift too many files round with this commit,
but at some point it would be nice to bring the sole `frontend` subproject
up to the root level.


http://notes.implicit.ly/post/147554513094/sbt-01312
http://www.scala-sbt.org/0.13/docs/Basic-Def.html
https://groups.google.com/d/msg/sbt-dev/S-pr2nZQ3KY/iX9q3cbrHVQJ
Copy link
Contributor

@AWare AWare left a comment

Choose a reason for hiding this comment

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

@prout-bot
Copy link

Overdue on PROD (merged by @rtyley 14 minutes and 3 seconds ago) What's gone wrong?

@rtyley
Copy link
Member Author

rtyley commented Oct 26, 2016

Overdue on PROD (merged by @rtyley 14 minutes and 3 seconds ago) What's gone wrong?

This deploy has failed due to sbt/sbt-native-packager#759 - it's affected two successive builds (10075 & 10077)

I notice that we seemed to find a fix for the issue with useNativeZip, I'll open a separate PR to make that change here.

rtyley added a commit that referenced this pull request Oct 26, 2016
Deployment of PR #1358 failed due to sbt/sbt-native-packager#759, where
jars in an artifact built with native packager occurred twice within the
zip file.

#1358 (comment)

On the recommendation of Nepomuk Seiler, we tried using `useNativeZip`
on a different project: guardian/members-data-api#100
...and it seemed to help.
@prout-bot
Copy link

Seen on PROD (merged by @rtyley 57 minutes and 30 seconds ago) Please check your changes!

@svillafe svillafe deleted the switch-to-build.sbt branch February 16, 2017 13:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants