Closed dark-steveneq closed 6 months ago
The update introduces a new method getContainerName()
to the MWCGroovyPlugin class, which simply returns a predefined constant NAME
. This enhancement is aimed at providing a straightforward way to access the container name, potentially improving code readability and maintainability by centralizing the container name definition.
File | Change Summary |
---|---|
.../mwc/groovyscript/MWCGroovyPlugin.java |
Added getContainerName() method returning NAME . |
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Seems like we worked on the exact same thing at the same moment XD Thanks anyway
Oh. Well that's a bit funny!
โฆin MWCGroovyPlugin
๐ Description
This PR fixes compile errors in
MWCGroovyPlugin
by implementinggetContainerName()
and removing@Override
forgetModName()
. Not sure if that's the right way to fix it (or if that's something on my end) but at least I can toy around with the code now.๐ฏ Goals
Make
MWCGroovyPlugin
compilableโ Non Goals
Make it a hack
๐ฆ Testing
If it compiles then everything's good
โฎ๏ธ Backwards Compatibility
๐ Related Issues & Documents
๐ผ๏ธ Screenshots/Recordings
๐ Added to documentation?
๐ [optional] What gif best describes this PR or how it makes you feel?
Summary by CodeRabbit