borkdude / tools

Tools
19 stars 2 forks source link

Recent change to antq script removed useful default behavior #10

Closed jdjohnston closed 1 month ago

jdjohnston commented 1 month ago

Currently, running antq without any arguments is equivalent to antq --help, but running clojure -X antq.tool/outdated without any arguments has the useful behavior of reporting any outdated dependencies. Sometimes it's useful to simply see if there are any outdated dependencies without actually making any changes to the deps.edn, project.clj, or such.

Before the most recent change, running antq.tool/outdated without arguments was the default behavior when runing the antq script without arguments.

Wouldn't most users be able to guess --help if needed. A help option is very common these days.

This is unrelated to the other part of the change to antq.clj, which was adding the dependency on org.slf4j/slf4j-nop. That seems good. I would simply prefer the old default behavior when the script is ran without arguments. Of course, it's your code :)

borkdude commented 1 month ago

Feel free to provide a PR

https://www.michielborkent.nl https://www.eetvoorjeleven.nu

On Wed, 18 Sep 2024 at 23:00, Jonathan D Johnston @.***> wrote:

Currently, running antq without any arguments is equivalent to antq --help, but running clojure -X antq.tool/outdated without any arguments has the useful behavior of simply reporting any outdated dependencies. Sometimes it's useful to simply see if there are any outdated dependencies without actually making any changes to the deps.edn, project.clj, or such.

Before the most recent change, running antq.tool/outdated without arguments was the default behavior when runing the antq script without arguments.

Wouldn't most users be able to guess --help if needed. A help option is very common these days.

This is unrelated to the other part of the change to antq.clj, which was adding the dependency on org.slf4j/slf4j-nop. That seems good. I would simply prefer the old default behavior when the script is ran without arguments. Of course, it's your code :)

— Reply to this email directly, view it on GitHub https://github.com/borkdude/tools/issues/10, or unsubscribe https://github.com/notifications/unsubscribe-auth/AACFSBRBJN6PXXUTRLDSGRDZXHSXJAVCNFSM6AAAAABOORBYF6VHI2DSMVQWIX3LMV43ASLTON2WKOZSGUZTINZSGM4DSMA . You are receiving this because you are subscribed to this thread.Message ID: @.***>

jdjohnston commented 1 month ago

@borkdude WIP. Repo forked, but I tend to be a bit slow & have limited Internet access.

borkdude commented 1 month ago

No worries, let me know if you need any help with anything

jdjohnston commented 1 month ago

PR accepted, so closing this Issue