search

clj-holmes / clj-watson

clojure deps SCA
Eclipse Public License 2.0
84 stars 9 forks source link

Address #103 properties file simplification #105

Closed seancorfield closed 3 months ago

seancorfield commented 3 months ago

This addresses the streamlining of the overridden properties to just those that need to be different.

This includes reverting the database location to the default (in the Maven cache).

This does not deprecate the -d option yet.

Signed-off-by: Sean Corfield sean@corfield.org

seancorfield commented 3 months ago

@lread Can you take a look at this and see if this looks sane? I wasn't sure how best to handle the deprecation of -d -- is that something that can be done within the new CLI lib you swapped in, or is that just a matter of documentation?

I forgot about the commit signing requirement -- I had to turn that off due to working with some stuff on Oracle Linux 8.8 locally -- so if this looks reasonable, I'll do a new PR with everything signed.

seancorfield commented 3 months ago

Handled in #106 so closing this out.