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

Updating to Grails 6.1.1 #521

Open
wants to merge 19 commits into
base: 5.0.x
Choose a base branch
from
Open

Updating to Grails 6.1.1 #521

wants to merge 19 commits into from

Conversation

jdaugherty
Copy link

Updating to Grails 6.1.1.

@jdaugherty
Copy link
Author

@jamesfredley this was my initial pass at merging my changes to support grails 6. Once we have an initial review done, I'll iterate on the build actions to get them working end-to-end.

@jdaugherty
Copy link
Author

Per discussion, the testapp-profile was used as part of the old travis CI testing. It looks like the test run there used to:

  1. Build the project & install into local maven
  2. Use grails profile to generate various test apps

Other than generating the grails apps, I can't see where those apps were ever used. I assume they were used for manual testing. It also looks like past github CI's only ever ran gradle check and published all of the gradle projects except spring-security-rest-testapp-profile.

Running gradle build at the root will treat the testapp-profile as a grails project - likely because grails-shell was originally removed in later versions of grails. This build will fail due to the same reason grails/grails-gradle-plugin#316 was created. For now, I'm going to remove the gradle project spring-security-rest-testapp-profile since we haven't tested with it since 2020 and because there are old secrets for APIs that aren't even valid anymore.

We can always restore this subproject later if we get profiles working correctly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

2 participants