[JDBC-88] Update :arglists metadata so it no longer confuses Eastwood Created: 13/Jan/14 Updated: 16/Jan/15
|Reporter:||Sean Corfield||Assignee:||Sean Corfield|
:arglists metadata should be valid Clojure syntax. Examples should be moved into docstrings.
|Comment by Andy Fingerhut [ 13/Jan/14 4:02 PM ]|
Eastwood's use of :arglists for :wrong-arity linter checking is definitely 'at odds' with the way java.jdbc and some other libraries use it for clearer documentation.
I would recommend leaving :arglists as-is for java.jdbc, and wait for the next version of Eastwood to add a capability to have something like an :eastwood-arglists for functions that explicitly replace their :arglists like this. Such a capability might best belong in Eastwood itself, since it is the new tool on the block.
As an example, I was considering that if a fn/macro explicitly chose to be Eastwood-friendly in this way, it could have an :arglists and :eastwood-arglists explicitly, for those few vars that replace their :arglists. When one is using Eastwood with a library that they do not want to modify in that way, there could be some extra config file for Eastwood where one could specify these :eastwood-arglists.
|Comment by Sean Corfield [ 28/Oct/14 6:47 PM ]|
Given Andy's comment back in January, lowering priority to Minor.
|Comment by Sean Corfield [ 16/Jan/15 1:14 PM ]|
Eastwood has recently added the ability to provide configuration to override specific function :arglists and out of the box comes with java.jdbc config to override the problematic functions: https://github.com/jonase/eastwood/blob/master/resource/eastwood/config/clojure-contrib.clj#L38-64
That means, as far as Eastwood is concerned, this is a non-issue, but it would be nice to fix those functions' metadata anyway.