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

#13552 - Merge gorm-docs into grails-data-mapping #1872

Merged
merged 228 commits into from
Jan 28, 2025

Conversation

jdaugherty
Copy link
Contributor

grails/grails-core#13552 Merge https://github.com/grails/gorm-docs into grails-data-mapping

Please note that I discovered that the gorm api docs were not published in 8.x. I have fixed this as of 9.x to resume publishing these to the "api" folder.

Look at the last commit to see the actual changes from the gorm-docs.

I temporarily added caffiene back to the gorm-test project since the groovy doc was failing. I'd like to get this merged, merge hibernate5, and merge mongodb then make a second pass at fixing the dependencies.

.github/workflows/gradle.yml Outdated Show resolved Hide resolved
.github/workflows/gradle.yml Show resolved Hide resolved
.github/workflows/gradle.yml Show resolved Hide resolved
docs/guide-developer/build.gradle Outdated Show resolved Hide resolved
@matrei
Copy link
Contributor

matrei commented Jan 28, 2025

The current state of root-website is not synced with https://gorm.grails.org

@jdaugherty
Copy link
Contributor Author

I need to rename root-website. I discovered that the source files in gorm-docs are being published afteralll: https://gorm.grails.org/latest/ ... just the main http://gorm.grails.org site is directly in gh-pages branch. We'll probably want to eventually move the main site into the project so it's published / versioned independently of the gh-pages branch. I will rename 'root-website' to data-mapping-website for now. I think we may want to remove it longer term and only have the root website.

docs/guide-rx/build.gradle Outdated Show resolved Hide resolved
@jdaugherty
Copy link
Contributor Author

@matrei Since I have made all of the requested changes, I am going ahead and merging this. If anything else needs changed, let me know and I will do it as part of the next review.

@jdaugherty jdaugherty merged commit 0a16122 into grails:9.0.x Jan 28, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants