spago icon indicating copy to clipboard operation
spago copied to clipboard

spago bundle-app overlapping short commands -p for --platform and --path

Open denizdogan opened this issue 3 years ago • 1 comments
trafficstars

spago bundle-app --help
Usage: spago bundle-app [-m|--main MAIN] [-t|--to TO] [-p|--platform PLATFORM]
                        [-y|--minify] [-s|--no-build] [-w|--watch]
                        [-l|--clear-screen] [-I|--allow-ignored]
                        [-p|--path PATH] [-x|--source-maps] [-n|--no-install]
                        [-u|--purs-args PURS-ARGS] [--deps-only] [--before ARG]
                        [--then ARG] [--else ARG]
  Bundle the project into an executable

Available options:
  -m,--main MAIN           Module to be used as the application's entry point
  -t,--to TO               The target file path
  -p,--platform PLATFORM   Bundle platform 'browser' (default) or 'node'
  -y,--minify              Minifies the bundle
  -s,--no-build            Skip build step
  -w,--watch               Watch for changes in local files and automatically
                           rebuild
  -l,--clear-screen        Clear the screen on rebuild (watch mode only)
  -I,--allow-ignored       Allow files ignored via .gitignore to trigger
                           rebuilds (watch mode only)
  -p,--path PATH           Source path to include (in addition to paths in
                           spago.dhall)
  -x,--source-maps         Whether to generate source maps for the bundle
  -n,--no-install          Don't run the automatic installation of packages
  -u,--purs-args PURS-ARGS Arguments to pass to purs compile. Wrap in quotes.
  --deps-only              Only use sources from dependencies, skipping the
                           project sources.
  --before ARG             Commands to run before a build.
  --then ARG               Commands to run following a successful build.
  --else ARG               Commands to run following an unsuccessful build.
  -h,--help                Show this help text

It seems that -p can mean both --path and --platform.

denizdogan avatar Sep 16 '22 07:09 denizdogan

Thanks for the report! We're in the middle of a rewrite, where we're going through all the flags and pruning some of the things. Will make sure to address this!

f-f avatar Sep 16 '22 08:09 f-f

The rewrite is now done, and the commands are not overlapping anymore, closing

f-f avatar Sep 20 '23 16:09 f-f