Dmitri Shuralyov
Dmitri Shuralyov
Can [`TestMain`](https://godoc.org/testing#hdr-Main) be used to assist with this?
> @shurcooL, please take a look. Sorry, but I don't have the bandwidth to properly review this change soon (maybe the coming weekend, but maybe not until the one after)....
Agreed, this should be improved so `gopherjs build` output shows the filename and line number information just like `go build` does.
I've looked into this, it turns out it's a regression caused by commit https://github.com/gopherjs/gopherjs/commit/642a7f3fafe4dc7c01179a830271aadd54641fdd which attempted to fix #119. I'll make a PR that resolves this issue. **Edit:** Done in...
> A common package could be referenced for all of the common code that ends up at the beginning of the output file right now. Right now DCE occurs based...
Not to take away form what you're saying, just trying to understand better, but doesn't the gopherjs build tool already do most of those things? When you do `gopherjs install`...
If you look at the [import graph](https://godoc.org/github.com/EngineerKamesh/igb/igweb/client?import-graph&hide=2) of that package, you'll see that it imports `go.isomorphicgo.org/go/isok`, which imports `github.com/tdewolff/minify/css`, which in turn imports `github.com/tdewolff/parse/v2`. It looks like one of the...
I can reproduce. This looks like a valid bug. Thanks for the report @shibukawa and for looking into it @theclapp. I think your analysis is correct.
@renfredxh just ran into this too. We worked around it, so it's not urgent, but fixing this will definitely be nice.
+1, I'm interested in this functionality too. Maybe I'll try to look into it soon.