Print a warning about SecurityManager being deprecated

groovy / GMavenPlus

A rewrite of GMaven, a Maven plugin for Groovy

Other

285 stars 35 forks source link

Print a warning about SecurityManager being deprecated #289

Closed keeganwitt closed 1 year ago

bmarwell commented 1 year ago

This PR has too much noise on unrelated lines. I cannot even see the actual change.

bmarwell commented 1 year ago

What about https://github.com/groovy/GMavenPlus/pull/287 ?

keeganwitt commented 1 year ago

What about #287 ?

Oops. Thank you for the reminder. I think I might have had local commits started and didn't see you opened a PR about that same time before opening this one. That would be the better solution. We probably should move forward with that one and not this one.

keeganwitt commented 1 year ago

Moved the unrelated changes into a separate PR: #290.