Skip to content

Conversation

@rainboyan
Copy link
Member

  • Refactor GrailsGradlePlugin and GrailsPluginGradlePlugin
  • Many improvements

- Use `tasks.register()` instead of `tasks.create()`
- Use `tasks.named()` instead of `tasks.getByName()`
- Remove unnecessary `@CompileDynamic`
- Use `tasks.register()` instead of `tasks.create()`
- Remove unnecessary `@CompileDynamic`
- Use `tasks.register()` instead of `tasks.create()`
- Remove unnecessary `@CompileDynamic`
- Remove `createConsoleTask()` and `createShellTask`
- Use `tasks.register()` instead of `tasks.create()`
- Change its return type to `void`
- Use `tasks.register()` instead of `tasks.create()`
- Make build properties ordered
- Use inline `ant.native2ascii()`
- Remove `createNative2AsciiTask`
- Enhance `buildClasspath()`
- Use `project.layout.buildDirectory` instead of `project.buildDir`
- Make `resolveClassesDirs()` private
…ension()`

- Change `configureGrailsBuildSettings()` return to `void`
- Migrate deprecated `project.buildDir`
- Add helper methods to `SourceSets`
- Use constant name of the Task and Configuration
- Use `tasks.register()` instead of `tasks.create()`
- Cleanup codes
@rainboyan rainboyan merged commit cd3486d into graceframework:2024.0.x Aug 28, 2025
2 checks passed
@rainboyan rainboyan deleted the refactor_gradle_plugins branch September 1, 2025 15:14
@rainboyan rainboyan self-assigned this Sep 1, 2025
@rainboyan rainboyan added type: enhancement A general enhancement theme: gradle Issues related to grade plugin labels Sep 1, 2025
@rainboyan rainboyan added this to the 2024.0.0-M2 milestone Sep 1, 2025
@rainboyan rainboyan moved this to Done in 2024.0.0-M2 Sep 1, 2025
@rainboyan rainboyan added the theme: refactor Issues related to Framework Code Refactoring label Oct 21, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

theme: gradle Issues related to grade plugin theme: refactor Issues related to Framework Code Refactoring type: enhancement A general enhancement

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

1 participant