Well its public so who knows in what other package is using it. And it's static, so who know what kind if monster is calling it from some inapropriate place. I have seen static methods being called from translation/localization files, because "it works".

[–] mbtrhcs 15 points 2 months ago* (last edited 2 months ago) (1 children)

IntelliJ finds most uses in my experience unless you're doing something weird with reflection or similar. And if it's a public facing API only used by the library's consumers..– it should be used in tests at the very least! Especially if it's prone to regressions like the comment suggests

[–] qaz@lemmy.world 13 points 2 months ago

There are only 7 unit test classes. 2 of which I wrote myself this month.

[–] cheddar@programming.dev 29 points 2 months ago (1 children)

Somebody should tell Albert and the others, they can let this method go.

[–] qaz@lemmy.world 29 points 2 months ago* (last edited 2 months ago) (2 children)

I already did so an hour ago

UPDATE: It has been merged

[–] AdNecrias@lemmy.pt 31 points 2 months ago

https://xkcd.com/1172

You know it's comming.

[–] cheddar@programming.dev 12 points 2 months ago

Alright, this is my first contribution to an open-source project, albeit indirect. I'll drink to that!