-
Notifications
You must be signed in to change notification settings - Fork 5
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 --analyze for dump pkgs info #25
base: master
Are you sure you want to change the base?
Conversation
I suppose that would work, although it would be better to output into a file on disk, rather than stdout. The way it currently is, the JSON data would get mixed with all the other outputs GopherJS may produce and separating them would be tricky. |
current start line with "anylize pkg: xxx" |
Sure, not saying it's not doable, only that it feels a bit inelegant. There's also another option, which is to make a better use of the source map GopherJS produces, add all relevant information into it and use it with any sourcemap visualizer tool like https://github.com/danvk/source-map-explorer. Having thought about it a bunch I increasingly feel like this would be both more elegant and versatile solution to the problem than this PR or mine. That said, any solution that exists and works is usually better than the most elegant solution that doesn't exist at all :) |
modify --anylize dump simply info, --anylize --json dump json info. |
gopherjs build --analyze for print simply pkg info. --anylize --json print pkgs json info.
for #3
> gopherjs --analyze hello
--analyze --json for print pkgs json format info.
> gopherjs build --analyze -json hello