[GitHub] [groovy] paulk-asert commented on pull request #1573: Move stream-related extension methods to StreamGroovyMethods

[GitBox <gi...@apache.org>]

paulk-asert commented on pull request #1573:
URL: https://github.com/apache/groovy/pull/1573#issuecomment-828824138


   One thing which we have found in the past when moving DGM methods is that some frameworks like Grails, Micronaut or Spock might use the DGM methods directly. They are public static methods after all even though we regard them as somewhat internal and expose them as methods "only" available through the runtime or when using @CS. So, we adopted the convention of marking old variants with @Deprecated which are ignored when loading DGM methods. Plugin DGM methods might be considered even slightly more internal since in theory they might not be activated on older JVM versions but they are still there as static methods, so it might pay to give them the same treatment. We tend to then delete those deprecated methods after a major version or two. For the same reason, I'd target only Groovy 4 for this change.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org