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

Cli with deno #59

Closed
wants to merge 1 commit into from
Closed

Cli with deno #59

wants to merge 1 commit into from

Conversation

sverhoeven
Copy link
Collaborator

@sverhoeven sverhoeven commented Oct 11, 2024

Refs #51

TODO

  • cli
  • compile
  • remove stuff done in CLI #57
  • happy inside monorepo
    • scripts in root
  • ci workflow to test cli and compiled executable
  • ci workflow on release to compile executable and add to release
  • improve usage message
  • improve error message
  • allow to use class package from workspace instead of from jsr
  • nice to haves
    • allow to read config from stdin
    • allow to write outout to stdout
    • add --help flag
    • add subcommand to validate config.json file
    • add subcommand to run experiment aka reference config and permutations configs
    • add output format csv or tsv instead of just json

@sverhoeven sverhoeven changed the base branch from main to 51-cli October 11, 2024 17:08
@sverhoeven
Copy link
Collaborator Author

sverhoeven commented Oct 14, 2024

Also try https://bun.sh/docs/bundler/executables

bun build src/cli.ts --compile --outfile class
ls -lnh class 
-rwxrwxrwx 1 1000 1000 97M Oct 14 10:55 class
./class 
Usage: class <config-file.json>

So bun works but executable is even bigger.

@sverhoeven
Copy link
Collaborator Author

As long as deno does not support "bin" in package.json this PR is not useful.

@sverhoeven sverhoeven closed this Oct 21, 2024
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