jamesfredley commented on code in PR #15500:
URL: https://github.com/apache/grails-core/pull/15500#discussion_r2956546156
##########
grails-core/src/main/groovy/grails/plugins/GrailsPluginManager.java:
##########
@@ -262,8 +260,8 @@ public interface GrailsPluginManager extends
ApplicationContextAware {
String getPluginPath(String name, boolean forceCamelCase);
/**
- * Looks up the plugin that defined the given instance. If no plugin
- * defined the instance then null is returned.
+ * Looks up the plugin that defined the given instance.
+ * If no plugin is defined the instance, then null is returned.
Review Comment:
minor, but maybe one of these:
- "If no plugin defined the instance, then null is returned." (keep original
active voice)
- "If no plugin is defined for the instance, then null is returned."
--
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]