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 support for the Bun and pnpm package managers. #231

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

commiterate
Copy link

@commiterate commiterate commented Jul 31, 2024

Description

Mirroring a similar contribution to shadow-cljs.

See the shadow-cljs issue for a more detailed "why".

This also adds support for node_modules indexing with the Bun runtime.

Testing

I've avoided adding additional tests in src/test/clojure/cljs/foreign/node_test.clj for the time being since I'm not sure how the ClojureScript CI environment is configured (not sure if Bun and pnpm are available).

Instead, I created a test repository with a barebones CLJS project using the fork (sourced in deps.edn with the git procurer).

https://github.com/commiterate/test-clojurescript-bun

This repository uses Emmy to create a Node.js application that just prints 21 (using Emmy's exp2 function) to stdout.

A reproducible shell is provided with Nix for additional testing by the ClojureScript team and others.


There's notes in the test repository's readme detailing some sharp edges with:

  • Using the ClojureScript compiler build API with tools.build (classpath issues breaking deps.cljs searches).
  • Using the ClojureScript compiler CLI.

These might be of interest to the ClojureScript team for future improvements.

@commiterate commiterate changed the title Add support for the Bun and pnpm package managers Add support for the Bun and pnpm package managers. Jul 31, 2024
@commiterate commiterate force-pushed the deps-cmd-add-bun-pnpm branch 3 times, most recently from 418d878 to d713463 Compare August 14, 2024 18:45
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.

1 participant