Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Revert "fix(core)!: Drop dependency grails-shell" #313

Merged
merged 2 commits into from
Aug 8, 2024

Conversation

codeconsole
Copy link
Contributor

Reverts #287 necessary to get IntelliJ grails plugin working again.

Do we want to bring back the profile plugins as well or just add

implementation "org.grails:grails-shell:$grailsVersion"

Are we deciding to bring back the old cli completely which could be run independently via ./grailsw ?

@jamesfredley
Copy link
Contributor

@codeconsole I found a reason to restore the grails-shell dependency here. Without this revert when you add org.grails.plugins:database-migration to your buildscript or buildSrc in order to run the dbm-* scripts, you end up having to manually add the grails-shell dependency to buildscript or buildSrc also, which was not needed before this was removed.

The profiles plugin will be needed until grails-shell and profiles are no longer used.

@codeconsole
Copy link
Contributor Author

@jamesfredley so are you recommending we release this into 6.2.1?

@jamesfredley
Copy link
Contributor

@codeconsole Yes, I think we should release this into 6.2.1.

@codeconsole codeconsole merged commit bca3bb7 into 6.2.x Aug 8, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working revert
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants