deadcode icon indicating copy to clipboard operation
deadcode copied to clipboard

Return nonzero exit code when issues are found

Open PLPeeters opened this issue 5 months ago • 4 comments

Fixes #20, #35

PLPeeters avatar Aug 29 '25 10:08 PLPeeters

Can this PR be landed? It seems like it would make the tool much more useful.

sbc100 avatar Sep 23 '25 18:09 sbc100

Can this PR be landed? It seems like it would make the tool much more useful.

Sure, I am going to create new release this week. It will include this fix.

albertas avatar Sep 24 '25 03:09 albertas

@albertas Don't want to be pushy, but I'm eagerly awaiting a release on this :)

PLPeeters avatar Oct 02 '25 12:10 PLPeeters

@albertas Don't want to be pushy, but I'm eagerly awaiting a release on this :)

We ended up use vulture for now since it does return and error code already: https://github.com/emscripten-core/emscripten/pull/25382. I'm not sure what the differences are, but they seem pretty similar.

sbc100 avatar Oct 02 '25 16:10 sbc100