apache / apache/grails-core

Elimination of Bootstrap Class

Open
#14,021 1 comment 0 reactions 0 assignees View on GitHub
relates-to: convention-over-configuration
Dominant language
Groovy
Stars
2.9k
Forks
975
Avg merge
1d 22h
Merged PRs (30d)
92

Description

### Issue description

As we have gone through the upgrade of Grails 7, we have noticed a lot of empty / default generated files. One of Grail's core principles is convention over configuration. The default, empty Bootstrap file is a good example of not adhering to this principle. However, for functional testing, we need to make sure there is test coverage for grails supported features.

We could remove the empty Bootstrap classes, and leave them in one spot with a comment on why an empty one exists. However, after investigating the changes to the Application class, Spring has taken an alternative approach. Spring uses the application class as a bootstrap class. I can't think of a reason why we couldn't do something similar.

This ticket would remove the Bootstrap.groovy support and integrate it into the Application class instance. This is a post 7.0 goal.

Contributor guide

Open the contributing guide

Research direction

Start by reviewing the existing Bootstrap.groovy support and the Application class changes described in the issue. Determine how supported features are functionally tested, then verify that Bootstrap integration is removed without losing coverage and that the application class provides the required bootstrap behavior.

Written by the indexing model from the issue text.

Assessment

Tech stack
groovy
Domain
backend
Issue type
Refactor
Difficulty
5/5
Estimated time
Over a week
Activity status
Stale
Clarity
Needs clarification
Newbie friendliness
25/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.