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

Refactor aliases to simplify config #61

Merged
merged 10 commits into from Feb 3, 2023

Conversation

practicalli-johnny
Copy link
Contributor

@practicalli-johnny practicalli-johnny commented Jan 30, 2023

Simplify the deps.edn configuration by moving alias definitions that are deprecated or unused to deps-deprecated.edm

Resolve #60

Deprecated projects

  • :project/jar & :project/uberjar - tools.build is recommended approach

Covered by other aliases

  • :repl/cider - covered by repl/interactive or repl/rebel (with rebel UI)
  • :format/cljfmt-check and :format/cljfmt-fix - covered by :format/cljfmt check | fix
  • :test/kaocha, :test/kaocha-global, :test/kaocha-watch, :test/kaocha-cljs - covered by :test/run, :test/watch

Simplified aliases

  • search/outdated contains excludes added for the deprecated Congnitect REBL aliases

Never used

  • :repl/prepl and :repl/prepl-cljs
  • :hack/antq - used once to fix an issue in antq
  • repl/rebel-reveal, repl/rebel-reveal-light, inspect/reveal, inspect/reveal-light, inspect/reveal-local, inspect/reveal-nrepl, inspect/reveal-nrepl-light, inspect/reveal-cider, inspect/reveal-cider-light - portal used rather than semi-commercial reveal
  • middleware/cider-clj, middleware/cider-clj-refactor, middleware/cider-cljs, middleware/nrebel, :lib/cider-nrepl
  • :lib/expectations, :lib/expectations-classic
  • :test/midje
  • :project/uberdeps
  • alpha/hotload-sockets

Not operational

  • :repl/rebel-figwheel - not working correctly

Simplify the alias definitions in deps.edn and remove unneccessary
sections

move unsued aliases definitions to deps-depricated.edn
- dev and test paths
- portal library for data inspection
- tools.deps.alpha add-libs3 Git repo for hotloading
- tools.namespace to reload namespaces
- slf4j-nop to suppress warnings
- test.check for generative testing
- ring-mock for testing web APIs in the REPL
Rebel terminal UI REPL with nREPL server, with dev and test tools and
libraries
- dev and test paths
- portal library for data inspection
- tools.deps.alpha add-libs3 Git repo for hotloading
- tools.namespace to reload namespaces
- slf4j-nop to suppress warnings
- test.check for generative testing
- ring-mock for testing web APIs in the REPL
@github-actions
Copy link

github-actions bot commented Jan 30, 2023

🦙 MegaLinter status: ⚠️ WARNING

Descriptor Linter Files Fixed Errors Elapsed time
✅ CLOJURE clj-kondo 2 0 0.1s
✅ MARKDOWN markdownlint 1 0 0.31s
✅ MARKDOWN markdown-link-check 1 0 14.78s
✅ MARKDOWN markdown-table-formatter 1 0 0.21s
⚠️ SPELL cspell 7 138 2.01s
✅ SPELL misspell 6 0 0.04s
⚠️ YAML prettier 3 1 0.41s
✅ YAML v8r 3 0 2.02s
✅ YAML yamllint 3 0 0.2s

See errors details in artifact MegaLinter reports on CI Job page
Set VALIDATE_ALL_CODEBASE: true in mega-linter.yml to validate all sources, not only the diff

MegaLinter is graciously provided by OX Security

@practicalli-johnny practicalli-johnny merged commit cd1c429 into live Feb 3, 2023
@practicalli-johnny practicalli-johnny deleted the refactor-aliases-simplify branch February 3, 2023 02:00
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.

refactor: deprecate unused aliases for simpler config
1 participant