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

convert filter definitions to taskKeys #157

Merged

Conversation

jrudolph
Copy link
Contributor

This will allow to load the filter definitions from something else than
source code without having to reload the project after having changed the
filters.

We are currently experimenting with moving the filters to external files so that the filters don't have to be compiled so often (IIRC big Seq(...) expressions can be quite costly for type inference). See akka/akka-http#677. If that approach works well it might make sense to move it here as well.

This will allow to load the filter definitions from something else than
source code without having to reload the project after having changed the
filters.
@ktoso
Copy link
Contributor

ktoso commented Dec 23, 2016

LGTM

@dwijnand
Copy link
Collaborator

dwijnand commented Feb 8, 2017

This might lead to #102 being fixed. Which would allow the Scala build to move to the sbt plugin rather than process forking to call MiMa's main.

@dwijnand dwijnand merged commit 92cbce5 into lightbend-labs:master Feb 8, 2017
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.

5 participants