matthiasblaesing commented on PR #6257: URL: https://github.com/apache/netbeans/pull/6257#issuecomment-1650499402
@asbachb I agree with your analysis in #6212, meaning, that there are version checks missing. I consider this the less invasive approach though, as it only affects only modified code sites. To identify locations to look at I used this procedure: - identify potentially affected modules: `grep -lr "isEjb3.*Supported" * | cut -f 1 --delimiter "/" | sort -u` - open all potentially affected modules and use the "Find Usages" search on the `isEjb3*` methods - review the calls The low hanging fruits were fixed, other locations need some more changes for example to annotation scanning. These classes were marked `@todo: Support JakartaEE`. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected] For further information about the NetBeans mailing lists, visit: https://cwiki.apache.org/confluence/display/NETBEANS/Mailing+lists
