Mathieu Barbin
Mathieu Barbin
:information_source: This PR is intended to be closed, as its changes are likely to be subsumed by PR #56 and any subsequent related PRs.
:information_source: This PR is intended to be closed, as its changes are likely to be subsumed by PR #56 and any subsequent related PRs.
By the way, something that didn't occur to me initially, if you prefer this can be merged into the existing flag, by making the cli ui `-unused-code-warnings=force` rather than adding...
Noting some action item for me to look into next on this PR: - [x] Rebase on top of #492 - [x] Rebase on top of #493 - [x] Extract...
@NathanReb thanks for #492 ! I like your proposed new tests a lot, and found it makes adding tests for this PR easier. By the way, do you like to...
Hi @NathanReb thanks so much for the heads up ! I am traveling at the moment and it would be difficult for me to allocate time to get it done...
@NathanReb One more thing while I am thinking about it : do as it is the most convenient for you, please do not worry about keeping or rebasing my commits,...
Hi @NathanReb just the heads up I found some time to make progress on the action items and pushed an update.
@NathanReb thank you for the review! > @mbarbin could you rebase on top of main? I'm actually thinking about dropping the `-unused-type-warnings` flag and instead disable all warnings through this...
> @mbarbin could you rebase on top of main? @NathanReb I removed the part about `-unused-type-warnings` and rebased on top of main.