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

Add 'asset-pipeline-gradle, grails-datastore-gorm-mongodb and views-gradle to grails-bom` #13872

Merged
merged 4 commits into from
Nov 22, 2024

Conversation

matrei
Copy link
Contributor

@matrei matrei commented Nov 22, 2024

Cleaning up dependency versions in grails-views, I noticed these versions are not in the BOM.
I see no reason why they should not be. They are all core dependencies.

@@ -25,6 +25,7 @@ ext {
// version name = versionNameOverride ?: artifactId-prefixes[0]
dependenciesOverview = """\
com.bertramlabs.plugins:asset-pipeline-grails::
com.bertramlabs.plugins:asset-pipeline-gradle::
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

you are not using SCOTL format. Shouldn't that be com.bertramlabs.plugins:asset-pipeline:gradle,grails: ? or is there a chance the version numbers won't match? I don't think they have ever been released independently.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is intentional. They have indeed been out of sync. There is for example no 4.4.0 version of asset-pipeline-grails

mattias@mattias-z4-linux:~/test/g622app$ mcs com.bertramlabs.plugins:asset-pipeline-grails:4*
Searching for com.bertramlabs.plugins:asset-pipeline-grails:4*...
Found 7 results

  Coordinates                                           Last updated
  ===========                                           ============
  com.bertramlabs.plugins:asset-pipeline-grails:4.5.2   07 Nov 2024 at 19:38 (CET)
  com.bertramlabs.plugins:asset-pipeline-grails:4.5.1   25 Jul 2024 at 18:00 (CEST)
  com.bertramlabs.plugins:asset-pipeline-grails:4.5.0   25 Jul 2024 at 17:11 (CEST)
  com.bertramlabs.plugins:asset-pipeline-grails:4.3.0   01 May 2023 at 18:20 (CEST)
  com.bertramlabs.plugins:asset-pipeline-grails:4.2.0   11 Apr 2023 at 17:36 (CEST)
  com.bertramlabs.plugins:asset-pipeline-grails:4.1.0   03 Mar 2023 at 17:57 (CET)
  com.bertramlabs.plugins:asset-pipeline-grails:4.0.0   03 Nov 2022 at 16:29 (CET)

mattias@mattias-z4-linux:~/test/g622app$ mcs com.bertramlabs.plugins:asset-pipeline-gradle:4*
Searching for com.bertramlabs.plugins:asset-pipeline-gradle:4*...
Found 8 results

  Coordinates                                           Last updated
  ===========                                           ============
  com.bertramlabs.plugins:asset-pipeline-gradle:4.5.1   25 Jul 2024 at 18:00 (CEST)
  com.bertramlabs.plugins:asset-pipeline-gradle:4.5.0   25 Jul 2024 at 17:11 (CEST)
  com.bertramlabs.plugins:asset-pipeline-gradle:4.4.0   26 Sep 2023 at 22:24 (CEST)
  com.bertramlabs.plugins:asset-pipeline-gradle:4.3.0   01 May 2023 at 18:20 (CEST)
  com.bertramlabs.plugins:asset-pipeline-gradle:4.2.0   11 Apr 2023 at 17:36 (CEST)
  com.bertramlabs.plugins:asset-pipeline-gradle:4.1.0   03 Mar 2023 at 17:57 (CET)
  com.bertramlabs.plugins:asset-pipeline-gradle:4.0.0   03 Nov 2022 at 16:29 (CET)
  com.bertramlabs.plugins:asset-pipeline-gradle:4.5.2   07 Nov 2024 at 19:38 (CET)

@@ -21,10 +21,11 @@ org.gradle.parallel=true
org.gradle.daemon=true
org.gradle.jvmargs=-Dfile.encoding=UTF-8 -Xmx1536M -XX:MaxMetaspaceSize=1024M

# Generated on Tue Oct 22 16:16:30 EDT 2024 by: ./gradlew :grails-bom:syncProps
# Generated on Fri Nov 22 09:58:56 CET 2024 by: ./gradlew :grails-bom:syncProps
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

+2 points for running the task 😉

@codeconsole codeconsole merged commit c797a7d into 7.0.x Nov 22, 2024
19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants