search

fangyidong / json-simple

A simple Java toolkit for JSON. You can use json-simple to encode or decode JSON text.
Apache License 2.0
746 stars 338 forks source link

Request: New Maven release for Junit dependency scope change #131

Open Macil opened 6 years ago

Macil commented 6 years ago

A commit 5 years ago (https://github.com/fangyidong/json-simple/commit/7ae9400664f8b2b43cbd704cf15af52d89b4c773#diff-600376dffeb79835ede4a0b285078036) fixed it so that json-simple only depends on Junit in the test scope, but there have been no releases since that fix. Could a new release be made at least for that?

I frequently work on projects that don't use Junit 4.10 but depend on libraries that depend on json-simple which then pulls in Junit 4.10, and then IDEs annoyingly auto-suggest Junit 4.10 annotations whenever we write tests even if we use a different test system. It's easy to pick a Junit 4.10 @Test annotation instead of our own test system's annotation and accidentally make a test that never runs without realizing it. It's a little silly issue since projects can be configured to exclude the Junit subdependency, but I see a lot of projects fail to do that, and the fix for them all has been sitting unused and ready for 5 years. I don't think you should worry that the scope change fix is too small for a release: it's better to have it out eventually, and there's no shortage of possible version numbers.

oscerd commented 4 years ago

I think a release is needed, just for leaving a definitive released without junit as runtime dep

chkpnt commented 3 years ago

+1

rcrlik commented 3 years ago

+1

Any update on this?

Lonzak commented 2 months ago

+1 Please release a new version. (or reflect the project status and set it to archive / not maintained anymore)