Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

alias: repl/reloaded-debug add sayid to reloaded workflow #75

Closed

Conversation

allentiak
Copy link
Contributor

📓 Description

A catch-all to have debugging, refactoring and hot reload.
Includes documentation.

@practicalli-johnny
Copy link
Contributor

Suggest using the alias name :repl/reloaded-debug

@allentiak
Copy link
Contributor Author

allentiak commented Aug 23, 2023 via email

`:repl/reloaded-debug` alias name seems to be a more consistent name as the
definition adds the sayid debug tool to the Practicalli REPL Reloaded workflow
@practicalli-johnny practicalli-johnny changed the title Add alias: rebel-debug-refactor-reload alias: repl/reloaded-debug adding flowstorm to reloaded workflow Sep 5, 2023
@practicalli-johnny practicalli-johnny changed the title alias: repl/reloaded-debug adding flowstorm to reloaded workflow alias: repl/reloaded-debug add sayid to reloaded workflow Sep 5, 2023
@practicalli-johnny
Copy link
Contributor

Adding this alias would define 3 aliases that include sayid and ideally I'd like to keep aliases to a minimum, especially as I do not use sayid and therefore unsure how to test it.

Which of the following aliases is the most useful to you?

  :repl/rebel-debug
  {:extra-deps {nrepl/nrepl                {:mvn/version "1.0.0"}
                cider/cider-nrepl          {:mvn/version "0.31.0"}
                com.billpiel/sayid         {:mvn/version "0.1.0"}
                com.bhauman/rebel-readline {:mvn/version "0.1.4"}}
   :main-opts  ["-e" "(apply require clojure.main/repl-requires)"
                "--main" "nrepl.cmdline"
                "--middleware" "[com.billpiel.sayid.nrepl-middleware/wrap-sayid,cider.nrepl/cider-middleware]"
                "--interactive"
                "-f" "rebel-readline.main/-main"]}

  :repl/rebel-debug-refactor
  {:extra-deps {nrepl/nrepl                   {:mvn/version "1.0.0"}
                cider/cider-nrepl             {:mvn/version "0.31.0"}
                refactor-nrepl/refactor-nrepl {:mvn/version "3.6.0"}
                com.billpiel/sayid            {:mvn/version "0.1.0"}
                com.bhauman/rebel-readline    {:mvn/version "0.1.4"}}
   :main-opts  ["-e" "(apply require clojure.main/repl-requires)"
                "--main" "nrepl.cmdline"
                "--middleware" "[com.billpiel.sayid.nrepl-middleware/wrap-sayid,refactor-nrepl.middleware/wrap-refactor,cider.nrepl/cider-middleware]"
                "--interactive"
                "-f" "rebel-readline.main/-main"]}

  :repl/reloaded-debug
  {:extra-paths ["dev" "test"]
   :extra-deps {nrepl/nrepl                  {:mvn/version "1.0.0"}
                cider/cider-nrepl            {:mvn/version "0.31.0"}
                com.bhauman/rebel-readline   {:mvn/version "0.1.4"}
                com.billpiel/sayid           {:mvn/version "0.1.0"}
                djblue/portal                {:mvn/version "0.42.1"}   ; portal data inspector
                clj-commons/clj-yaml         {:mvn/version "1.0.26"}   ; portal yaml support (optional)
                org.clojure/tools.namespace  {:mvn/version "1.4.4"}
                org.clojure/tools.trace      {:mvn/version "0.7.11"}
                org.slf4j/slf4j-nop          {:mvn/version "2.0.7"}
                com.brunobonacci/mulog       {:mvn/version "0.9.0"}
                lambdaisland/kaocha          {:mvn/version "1.85.1342"}
                org.clojure/test.check       {:mvn/version "1.1.1"}
                refactor-nrepl/refactor-nrepl {:mvn/version "3.6.0"}
                ring/ring-mock               {:mvn/version "0.4.0"}
                criterium/criterium          {:mvn/version "0.4.6"}}
   :main-opts  ["-e" "(apply require clojure.main/repl-requires)"
                "--main" "nrepl.cmdline"
                "--middleware" "[cider.nrepl/cider-middleware,refactor-nrepl.middleware/wrap-refactor,com.billpiel.sayid.nrepl-middleware/wrap-sayid,portal.nrepl/wrap-portal]"
                "--interactive"
                "-f" "rebel-readline.main/-main"]}

I'm also considering deprecating the following aliases and replacing them with what ever rebel versions are actually of use.

  :repl/refactor
  {:extra-deps {nrepl/nrepl                   {:mvn/version "1.0.0"}
                cider/cider-nrepl             {:mvn/version "0.31.0"}
                refactor-nrepl/refactor-nrepl {:mvn/version "3.6.0"}}
   :main-opts  ["--main" "nrepl.cmdline"
                "--middleware" "[refactor-nrepl.middleware/wrap-refactor,cider.nrepl/cider-middleware]"
                "--interactive"]}

  :repl/debug
  {:extra-deps {nrepl/nrepl        {:mvn/version "1.0.0"}
                com.billpiel/sayid {:mvn/version "0.1.0"}
                cider/cider-nrepl  {:mvn/version "0.31.0"}}
   :main-opts  ["--main" "nrepl.cmdline"
                "--middleware" "[com.billpiel.sayid.nrepl-middleware/wrap-sayid,cider.nrepl/cider-middleware]"
                "--interactive"]}

  :repl/debug-refactor
  {:extra-deps {nrepl/nrepl                   {:mvn/version "1.0.0"}
                cider/cider-nrepl             {:mvn/version "0.31.0"}
                refactor-nrepl/refactor-nrepl {:mvn/version "3.6.0"}
                com.billpiel/sayid            {:mvn/version "0.1.0"}}
   :main-opts  ["--main" "nrepl.cmdline"
                "--middleware" "[com.billpiel.sayid.nrepl-middleware/wrap-sayid,refactor-nrepl.middleware/wrap-refactor,cider.nrepl/cider-middleware]"
                "--interactive"]}

@allentiak
Copy link
Contributor Author

I'm sorry for my late reply.
I currently do not have the time for testing this.
I will get back to this as soon as I do.

@allentiak allentiak deleted the rebel-debug-refactor-reload branch November 15, 2023 11:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants