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

Run plugin only during test runs, and initialize only once #8

Open
crisptrutski opened this issue Jan 4, 2015 · 0 comments
Open

Run plugin only during test runs, and initialize only once #8

crisptrutski opened this issue Jan 4, 2015 · 0 comments

Comments

@crisptrutski
Copy link
Collaborator

Think the best way to spare unnecessary is to use the add-hook to intercept midje tasks, and skip adding middleware.

Additionally when running lein midje :autowatch from Instar, I noticed that the existing middleware function is actually firing three times. This is bad enough when just running eval-in-project and doing IO, but when multiple filesystem watchers are being created the cost becomes more noticeable.

Should be able to wrap the initialization for file watcher in a "execute at most once" to avoid this sort of issue occurring again (although fairly benign - just inefficient)

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

No branches or pull requests

1 participant