-
Notifications
You must be signed in to change notification settings - Fork 19
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
NPM Release #40
NPM Release #40
Conversation
zampino
commented
Oct 31, 2023
•
edited
Loading
edited
- adjust for NPM release
- update usage in README and landing page
- add squint build snapshot with import-from-CDN demo
README.md
Outdated
@@ -35,6 +35,25 @@ Enabling a decent Clojure/Script editor experience in the browser. Built for and | |||
{:deps {io.github.nextjournal/clojure-mode {:git/sha "<SHA>"}}} | |||
``` | |||
|
|||
### Use if from [NPM](https://www.npmjs.com/package/@nextjournal/clojure-mode) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Use if?
I tried using this today, and I'm getting the error: (!) Plugin node-resolve: Could not resolve import "@nextjournal/clojure-mode" in /Users/sam/sstraust/newsViewer/newsjs/resources/codemirror_editor.js using exports defined in /Users/sam/sstraust/newsViewer/newsjs/resources/node_modules/@nextjournal/clojure-mode/package.json. Invalid mapping: "dist/nextjournal/clojure_mode.mjs". (I'm new in this area, so I could be missing something obvious) |
@sstraust Is it possible for you to put out a github repo with a repro of your problem? |
Perhaps this is the bug: @sstraust Could you try editing |
yes! adding ./ in the suggested place removed the error warning from my build |
I've published 0.2.0 on npm. Please let us know if that resolves your issue, thanks! |
yep! it now works for me! Thanks for the speedy fix! (I edited package.json to use "^0.2.0", then ran:
) compilation ran with no warnings, and I am able to see the editor in my browser. |