Skip to content

Extract Def.settings from Project#settings.#2151

Merged
jsuereth merged 2 commits intosbt:0.13from
dwijnand:add-Def.settings
Aug 25, 2015
Merged

Extract Def.settings from Project#settings.#2151
jsuereth merged 2 commits intosbt:0.13from
dwijnand:add-Def.settings

Conversation

@dwijnand
Copy link
Member

This allows for the same functionality that using SettingsDefinition in
Project#settings allows (specifying either bare Setting[_] or a Seq[Setting[_]])
to be available outside of the settings for a project, for instance when
defining a val.

In short, it allows:

val modelSettings = Def.settings(
  sharedSettings,
  libraryDependencies += foo
)

@typesafe-tools
Copy link

Can one of the admins verify this patch?

@eed3si9n
Copy link
Member

Notes please.
LGTM pending Travis.

@dwijnand dwijnand self-assigned this Aug 17, 2015
@eed3si9n
Copy link
Member

Could you rebase this? The Merge button is gone.

This allows for the same functionality that using SettingsDefinition in
Project#settings allows (specifying either bare Setting[_] or a Seq[Setting[_]])
to be available outside of the settings for a project, for instance when
defining a val.

In short, it allows:

    val modelSettings = Def.settings(
      sharedSettings,
      libraryDependencies += foo
    )
@dwijnand
Copy link
Member Author

Done.

@jsuereth
Copy link
Member

Nice cleanups :)

jsuereth added a commit that referenced this pull request Aug 25, 2015
Extract Def.settings from Project#settings.
@jsuereth jsuereth merged commit da8bb5b into sbt:0.13 Aug 25, 2015
@dwijnand dwijnand deleted the add-Def.settings branch August 25, 2015 13:53
@dwijnand dwijnand removed their assignment Jan 13, 2017
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.

4 participants