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

Add a date formatting option to cljsbuild output #419

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

danielcompton
Copy link
Contributor

This is still WIP, but I wanted to publish it to show its relevance to #418

Fixes #397

@danielcompton
Copy link
Contributor Author

After seeing #418, I wonder if there needs to be a set of default loggers set in support that can be used by any project consuming just the support library, and plugin can override these loggers and install Leiningen loggers which will respect the LEIN_SILENT option.

@mneise
Copy link
Collaborator

mneise commented Oct 11, 2015

This sounds like a really good idea and would save us from adding the extra leiningen-core dependency to the core project 👍

@tmarble
Copy link

tmarble commented Oct 13, 2015

@danielcompton are you planning to work on this?

@danielcompton
Copy link
Contributor Author

Yep, definitely working on it.

@tmarble
Copy link

tmarble commented Nov 19, 2015

@danielcompton just want to check in... would you like me to take a try at this?
See also #397 #417 #418

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