Accept symbols for 'require' and 'use' inside 'ns'

Description

clojure.core.ns allows much more syntactic variation than its docstring describes.

One example is (ns foo (require bar)) where require should be the keyword :require.

As of 0.2.9, tools.namespace silently ignores these forms.

Environment

None

Activity

Show:
Andy Fingerhut
December 22, 2014, 7:34 PM

Code change and test look perfect to me.

import
December 23, 2014, 5:39 PM

Comment made by: petrglad

Line 48 in 0001-TNS-32-handle-non-keyword-clause-heads-in-ns-form.patch:
"(def t-clauses-without-keywords" -> "(deftest t-clauses-without-keywords" ?

Andy Fingerhut
December 23, 2014, 6:35 PM

Good catch, Petr. I missed that.

Stuart Sierra
January 30, 2015, 3:42 PM
Stuart Sierra
February 1, 2015, 3:39 PM

Included in release version 0.2.9

Completed

Assignee

Stuart Sierra

Reporter

Stuart Sierra

Labels

None

Approval

None

Patch

Code and Test

Priority

Minor