ppxlib icon indicating copy to clipboard operation
ppxlib copied to clipboard

Add an -unused-type-warnings flag to the driver

Open NathanReb opened this issue 1 year ago • 2 comments

This allows disabling the generation of let _ = fun (_ : t) -> () strucutre items for each type using derivers.

The feature was meant to automatically disable warning 34 when using [@@deriving ...].

This is based on top of #492 to ease testing for the feature, only the last commit is relevant.

NathanReb avatar May 29 '24 09:05 NathanReb

CC @mbarbin, if you want to test this feature.

CC @ceastlund, I'd be curious to know if you get any warnings when enabling this flag for JS' codebase.

NathanReb avatar May 29 '24 09:05 NathanReb

I created a preview package that includes this change, and tested it in a PR in a project that could benefits from that. I was able to remove my hack and coverage stayed at 100%. :+1:

mbarbin avatar May 30 '24 09:05 mbarbin

@patricoferris We were discussing with @NathanReb and I've made an attempt to implement this feature on top of another PR, see #511 . I am not sure which matches best what Nathan had in mind, just giving you the heads up.

mbarbin avatar Jul 16 '24 07:07 mbarbin

Yes this PR is a bit out of date now. Since @mbarbin started working on the right implem, I'd prefer to take his contributions! Let's close this!

NathanReb avatar Jul 16 '24 08:07 NathanReb