practicalli / clojure-cli-config Goto Github PK

Practicalli has used SuperLinter to lint markdown content for books. Add this to practicalli/clojure-deps-edn project to verify the markdown in the readme.md file, describing the project in detail.

After following the installation instructions, calling any of the provided aliases failed with the error message "No function found on command line or in :exec-fn". Due to the help of Alex Miller and Sean Corfield on the Clojurians Slack channel, it became clear that the issue is that after installing the Clojure CLI tools, my config-user is not pointing to $HOME/.clojure but to $HOME/.config/clojure. I.e., see the output of clojure -Sdescribe:

[host->~]clojure -Sdescribe
{:version "1.10.1.727"
 :config-files ["/usr/local/lib/clojure/deps.edn" "/home/schauer/.config/clojure/deps.edn" ]
 :config-user "/home/schaueho/.config/clojure/deps.edn"
 :config-project "deps.edn"
 :install-dir "/usr/local/lib/clojure"
 :config-dir "/home/schaueho/.config/clojure"
 :cache-dir "/home/schaueho/.config/clojure/.cpcache"
 :force false
 :repro false
 :main-aliases ""
 :repl-aliases ""}

According to Alex, the reason is in my case that the environment variable $XDG_CONFIG_HOME points to /home/schaueho/. Apparently, setting $CLJ_CONFIG could lead to similar effects, as described in deps and cli documentation.

My proposal would be to change the documentation so that instead of just assuming ~/.clojure/ to specifically mention the value of :config-user from the output above.

Latest clojure, latest clojure-deps-edn.

$ clojure -T:search/outdated
No function found on command line or in :exec-fn

$ clojure -version
Clojure CLI version 1.11.1.1149

$ cd ~/.clojure
[collider@grabber .clojure]$ git status 
On branch live
Your branch is up to date with 'origin/live'.

nothing to commit, working tree clean

Design an alias for the https://github.com/fabiodomingues/clj-depend tool for analysing dependencies

Hey there!

Thanks for your project. I'm experimenting with using it as my source of truth for all sorts of CI checks i.e. not just for personal productivity.

I noticed a couple possible issues with the clj-kondo setup:

• :main-opts ["-m" "clj-kondo.main" "--lint" "src"]} does not specify "test". This seems a dangerous absence to have by default. It would seem sensible to either drop src or to also include test.
• For best results, clj-kondo shouldn't be run as a one-liner, see e.g. https://github.com/clj-kondo/clj-kondo#project-setup. So it seems to me that there should be an additional alias?
  • The usage I would foresee is clojure -M:lint/clj-kondo.analyse && clojure -M:lint/clj-kondo src test

wdyt?

Cheers - V

Currently, Updating practicalli/clojure-deps-edn links to https://github.com/practicalli/clojure-deps-edn/blob/live/updating-practicalli/clojure-deps-edn which does not exist.

It looks like it should link to updating-practicalliclojure-deps-edn (without the /).

Per the changes referenced in this issue, with your approval, I'd like to work on a pull request to change the documentation and language in your deps.edn to reflect changes to the hot-load alias.

Hi, @practicalli-john -

Thanks for making all these useful aliases available to the Clojure community. I think it might be helpful for myself and others if you clarified what license you're making this project available under, as there currently doesn't seem to be a license as far as I can tell.

Currently the readme.md says to execute: clojure -X:deps mvn-install project.jar

The result I get is:
Key is missing value: project.jar

When I execute clojure -X:deps mvn-install :jar project.jar I get

my-library $ clojure -X:deps mvn-install :jar "project.jar"
Installing project.jar
Execution error (IllegalArgumentException) at clojure.tools.cli.api/read-pom-in-jar (api.clj:310).
No implementation of method: :as-file of protocol: #'clojure.java.io/Coercions found for class: clojure.lang.Symbol

I had a setup for debugging some hairy recursive code using Reveal (no reveal specific code required in my app).

Now I return a month or so later and it's gone? diff here.

How do you evaluate that something is 'unused'? Can you keep metrics on clojure-deps-edn alias downloads?

With Portal I seem to have to launch an inspector from within the app. Extra steps for me given it's HumbleUI and can crash if I am doing crazy things.

Also it feels very un-Clojure like that these aliases are mutable and removed (thinking of Rich Hickey's talk and the blessings of Maven immutability).

My code tap>s out reams of info, which was very smooth scrolling through in Reveal (order is important for analysis). Perhaps I can get Portal to the same place.. but will require investigative work and so is jarring.

Thanks for the project and looking forward to your thoughts on this issue.

Move all alias definitions that are never used to a different file, keeping the deps.edn file much simpler to work with.

Keep the deprecated alias definitions should they be of interest to the community

Add a GitHub action for mega-linter that checks markdown, yaml and deps.edn files.

Run antq and update library versions within aliases

GitHub action report:
https://github.com/practicalli/clojure-cli-config/actions/runs/6045350640/job/16405274396#step:8:1

The :mvn/local-repo was changed to ".cache/maven/repository" which is relative ( this probably should be absolute path in $HOME/ )
It appears to break the existing usage.
I commented it out for now - to default back to ~/.m2

I've just updated some of my repos to note that the user-level deps.edn file is in ~/.config/clojure/ on some systems (the deps/CLI reference notes them as "Freedesktop conventions"). This seems to be where the XDG_CONFIG environment variable points on such systems (and that takes precedence over the HOME variable in clojure shell script, if CLJ_CONFIG is not set).

Just in case you want to add a note in your repo.

'user-config' is now 'config-user'

`samusz@CESP-176-39 polylithExample % clojure -Sdescribe

{:version "1.10.3.833"
:config-files ["/usr/local/Cellar/clojure/1.10.3.833/deps.edn" "/Users/samusz/.clojure/deps.edn" "deps.edn" ]
:config-user "/Users/samusz/.clojure/deps.edn"
:config-project "deps.edn"
:install-dir "/usr/local/Cellar/clojure/1.10.3.833"
:config-dir "/Users/samusz/.clojure"
:cache-dir ".cpcache"
:force false
:repro false
:main-aliases ""
:repl-aliases ""}`

After running a test suite and reporting the failures it just hangs until you kill the process. eftest.runner/run-tests is intended for REPL usage and returns a map with a summary and doesn't initiate shutdown. If you look at the source for lein-eftest it checks the number of failures/errors returned by run-tests and exits with nonzero if they are nonzero -- see https://github.com/weavejester/eftest/blob/master/lein-eftest/src/leiningen/eftest.clj#L83-L116

An alias like below starts CIDER-nREPL and opens Clerk and starts its "tap inspector", which is conceptually the same as portal and reveal, just powered by Clerk.
After this any "tapped" value is rendered in Browser.

I noticed an "issue", by "tap>" not giving the control back to the repl...
This does not happen, if you connect Emacs or an other nRepl client.

Not sure, what the root cause of this is

:inspect/clerk
  {:extra-deps {io.github.nextjournal/clerk {:mvn/version "0.15.957"}
                nrepl/nrepl                   {:mvn/version "1.0.0"}
                cider/cider-nrepl             {:mvn/version "0.28.7"}
                refactor-nrepl/refactor-nrepl {:mvn/version "3.6.0"}}
   :main-opts ["-e" "(require '[nextjournal.clerk])(nextjournal.clerk/show! 'nextjournal.clerk.tap)(nextjournal.clerk/serve! {:browse? true})"
               "-m" "nrepl.cmdline"
               "--interactive"
               "--middleware" "[refactor-nrepl.middleware/wrap-refactor,cider.nrepl/cider-middleware]"]}

The same command line args as applied to repl/remote do not work with repl/rebel-remote.
--host and --port do not have any effect.

:replace-deps aliases are not being updated by antq using the :project/outdated alias.

Example
https://clojars.org/uberdeps
clj-new

Update: issue reported on the antq project liquidz/antq#44

Add a workable approach to using tools.build from Practicalli Clojure deps.edn, ideally without having to add any other files to a project (external dependency to using the alias)

Putting this on hold as tools.build seems more applicable to a project deps.edn than user config

References

• tools.build guide
• https://github.com/liquidz/build.edn -

Background

:project/jar and :project/uberjar aliases depend on depstar which was deprecated (and archived) for six months.

tools.build should be the preferred approach

https://github.com/vvvvalvalval/scope-capture

It would probably be good to update the comments

When creating a new service from the template I get an error for io.github.seancorfield/deps-new stating that the tag (0.6.0) is invalid.

Changing the tag for io.github.seancorfield/deps-new in deps.edn to 'v0.6.0' fixes the issue.

I will submit a pull request.

The :lib/sayid library seems to be of little use without the associated middleware, so that alias should be deprecated

clojure-emacs/sayid#35

Related: Update Practicalli Spacemacs book - https://practical.li/spacemacs/debug-clojure/sayid-debug/

I run the new project template and it flunks.

user$ clojure -X:project/new :template app :name practicalli/testapp
No function found on command line or in :exec-fn

Here's the output of clojure -Sdescribe

{:version "1.10.1.763"
 :config-files ["/usr/share/clojure/deps.edn" "/home/billy/.clojure/deps.edn" ]
 :config-user "/home/billy/.clojure/deps.edn"
 :config-project "deps.edn"
 :install-dir "/usr/share/clojure"
 :config-dir "/home/billy/.clojure"
 :cache-dir "/home/billy/.clojure/.cpcache"
 :force false
 :repro false
 :main-aliases ""
 :repl-aliases ""}

The repository has evolved to be more than just a deps.edn file and is specifically referred to as a Clojure CLI configuration.

use the consistent branch name main as the default branch

• rename the repository to clojure-cli-config (github will redirect the old name)
• rename the live branch to main
• update description in readme.md file

Please raise an issue before raising a pull request in order to avoid working on something that would not be accepted.

Sweeping changes to this repository (or any open source project) are less likely to be accepted and take longer to review and test by the maintainers.

For example, updating a number of different libraries all at once will require a lot of manual testing to ensure each alias still works, especially with some of the recent changes in the ClojureScript library.

Pull requests that change deprecated aliases will be rejected. Those aliases are to be removed, so no further testing will be done. Updating deprecated aliases does not demonstrate an understanding of the project concerns.

In the recent change below, clone was removed, so this isn't a valid git command any more. Also, I don't know if it's just me or all Mac users, but $XDG_CONFIG_HOME isn't set at all, FWIW.

- git clone your-fork-url ~/.clojure/
+ git [email protected]:practicalli/clojure-deps-edn.git $XDG_CONFIG_HOME/clojure