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

Allow multiple repl task profiles #1704

Merged

Conversation

hugoduncan
Copy link
Contributor

The :repl profile does not compose well when specified in both
project.clj and profiles.clj files.

Merge all profiles with ^:repl metadata in the repl task. This allows
multiple profiles to be applied for the repl task.

Note that this could be done for other task specific profiles (e.g. test).

Preserve any metadata on profile maps specified in project.clj files.
Function to return a sequence of profile keywords for all project
profiles with metadata matching a predicate.
The :repl profile does not compose well when specified in both
project.clj and profiles.clj files.

Merge all profiles with ^:repl metadata in the repl task.  This allows
multiple profiles to be applied for the repl task.
@technomancy
Copy link
Owner

Sorry for the delay here. I like the idea behind this; it definitely makes the specification more flexible. Can you confirm that this would work for user-specified :repl profiles? It's not clear to me whether the metadata would be applied automatically in that case.

@technomancy
Copy link
Owner

Just tested to confirm that the case in question above works, so this looks good.

technomancy added a commit that referenced this pull request Oct 12, 2014
@technomancy technomancy merged commit fa64351 into technomancy:master Oct 12, 2014
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.

None yet

2 participants