Permalink
Commits on Jan 25, 2017
  1. Remove lein-newnew references from templates docs

    gfredericks committed Jan 25, 2017
Commits on Jan 18, 2015
Commits on Jul 7, 2014
  1. Fix the test that's been failing on Travis

    gfredericks committed Jul 7, 2014
    The problem was something to do with the project map being too
    bare. Switching to `defproject` fixes it.
Commits on Jun 13, 2014
Commits on Jun 11, 2014
Commits on Mar 14, 2014
  1. lein run: don't catch app errors as require errors

    gfredericks committed Mar 14, 2014
    closes #1469
    
    `lein run` currently has some special error handling around
    FileNotFoundException which helps it decide what exactly it should
    do. Unfortunately if the actual running of the given main function
    itself throws a FileNotFoundException this is currently caught by
    Leiningen and assumed to be a require error.
    
    The simple solution is to run the main method outside of that error
    handling block.
    
    Regression test provided.
Commits on Mar 11, 2014
Commits on Mar 8, 2014
  1. Don't call concat

    gfredericks committed Mar 8, 2014
    There, now there is slightly less code.
Commits on Aug 5, 2013
  1. Issue #1276: cleanup wrong-arg-num err msg on 0-arg tasks

    gfredericks committed Aug 5, 2013
    0 arg task arglist was printing as `""` since passing `[project]` to
    `next` resulted in `nil` which prints as `""`. Used `(comp vec next)`
    instead to force it into `[]` which prints as `"[]"`.
Commits on Aug 1, 2013
  1. #1269: Skip tests differently when using :test-selectors

    gfredericks committed Aug 1, 2013
    The old method skipped tests by adding a hook to
    clojure.test/test-var, the problem being that :each fixtures
    associated with the test have already run at that point, which is
    unideal.
    
    This change skips test by removing their :test metadata before running
    the tests at all, which causes clojure.test to not see it as a test.
    
    We use ns-interns to enumerate the vars, the same way clojure.test
    does.
  2. Refactor a couple reduces into a couple somes

    gfredericks committed Aug 1, 2013
    I think it's a little simpler this way.
Commits on Jun 4, 2013
  1. Crash on :aot [does.not.exist]; also regression test; Fixes #1205

    gfredericks committed Jun 4, 2013
    We want missing namespaces in :aot to not be silently ignored.
    Implemented by removing the file-existence check from the function
    that determines which namespaces need to be compiled.
    
    This means we won't get the error if you compile an existing namespace
    and subsequently delete the *.clj file. Oh well.
Commits on May 23, 2013
  1. Add back support for java classes in :main

    gfredericks committed May 23, 2013
    This commit bends over backwards to add back support for java
    classes in :main that was removed in 71e609c. The crux of
    the complexity was that when we try to require the namespace
    relevant to the argument to `lein run`, we could get a
    FileNotFoundException for two distinct reasons -- either the
    namespace we were trying to run doesn't exist, in which case
    we want to check if it was actually a class name; or the
    namespace exists but it or some other dependency is trying
    to require a namespace that doesn't exist. In the latter case
    we definitely want the FileNotFoundException to propagate up
    to the top. But distinguishing these two cases by examining
    the exception is hacky at best, so the alternative adopted
    here is to hold on to the exception, check if a class exists,
    and if not, re-throw the exception.
    
    This should handle the majority of use cases correctly. The
    test added in 71e609c is still relevant and still passes.
Commits on May 22, 2013
  1. Don't try static methods on `lein run`: better error msg

    gfredericks committed May 22, 2013
    lein run tries to support invoking static methods on classes by
    swallowing FileNotFoundExceptions when requiring the namespace to run,
    which can obscure the real error when there's a real issue loading a
    namespace.
    
    Phil said he wasn't interested in keeping the (undocumented) feature
    around, so the easiest fix to the error message problem was to remove
    that feature.
    
    Added a test as well.