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

Add Calva Connect Sequence #8

Closed
wants to merge 70 commits into from
Closed

Add Calva Connect Sequence #8

wants to merge 70 commits into from

Conversation

PEZ
Copy link

@PEZ PEZ commented Feb 14, 2023

When using Calva users sometimes have troubles getting the right dependencies in place. For this we usually recommend using Calva Jack-in, which starts the REPL, injecting the dependencies that Calva needs.

This PR makes starting the app and connecting Calva a short process like so:

  1. Open the project in a VS Code window.
  2. Issue the VS Code command: Calva: Start a Project REPL and Connect (aka Jack-in)
  3. In the Project type menu, choose Electric Starter App
  4. Soon the Calva Output window will print 👉 App server available at http://0.0.0.0:8080

Without this PR the process is:

  1. Open the project in a VS Code window.
  2. Issue the VS Code command: Calva: Start a Project REPL and Connect (aka Jack-in)
  3. In the Project type menu, choose shadow-cljs
  4. In the Launch build menu, choose :app
  5. In the Connect build menu, choose :app
  6. Open src/user.clj and load it in the REPL (ctrl+alt+c enter)
  7. Evaluate the (main) form at the bottom of the file
  8. The Calva Output window will print 👉 App server available at http://0.0.0.0:8080

I think the README should have instructions for starting the project in the editor. But as there was nothing about this there, it is maybe weird to only have Calva instructions? I could always add it, and other people can add sections for other editors. Let me know what you think.

filipesilva and others added 30 commits May 14, 2022 23:48
(cherry picked from commit 979a191)
(cherry picked from commit deacc09)
Replit will add `Content-Type:text/plain` header if no Content-Type header is
present. This prevents CSS to load, because browsers expect `text/css`, not
`text/plain`.
dustingetz and others added 26 commits March 9, 2023 10:23
…ygiene and avoid cljs.env dependency on server at runtime
AOT compilation was triggering the top level check for
HYPERFIDDLE_ELECTRIC_SERVER_VERSION

Updated readme accordingly
progress=plain makes docker build more verbose
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.

None yet

9 participants