Skip to content

Update contribution link and clarify encrypted field configuration. #4991

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

ricardohsmello
Copy link

Updated README.adoc to reflect the current contribution process, replacing the outdated CLA link with the new Developer Certificate of Origin (DCO) instructions.

In mongo-encryption.adoc, updated the code sample to include the required keyId(...) call when configuring a field with Queryable Encryption.

  • You have read the Spring Data contribution guidelines.
  • You use the code formatters provided here and have them applied to your changes. Don’t submit any formatting related changes.
  • You submit test cases (unit or integration tests) that back your changes.
  • You added yourself as author in the headers of the classes you touched. Amend the date range in the Apache license header if needed. For new types, add the license header (copy from another file and set the current year only).

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jun 3, 2025
README.adoc Outdated
@@ -211,8 +211,9 @@ Now you are ready to build Spring Data MongoDB. Simply enter the following `mvnw

If you want to build with the regular `mvn` command, you will need https://maven.apache.org/run-maven/index.html[Maven v3.8.0 or above].

_Also see link:CONTRIBUTING.adoc[CONTRIBUTING.adoc] if you wish to submit pull requests, and in particular, please sign
the https://cla.pivotal.io/sign/spring[Contributor’s Agreement] before your first non-trivial change._
_Also see link:CONTRIBUTING.adoc[CONTRIBUTING.adoc] if you wish to submit pull requests.
Copy link
Member

Choose a reason for hiding this comment

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

Removing the Link to cla.pivotal.io is good, we have further details outlined on DCO at https://github.com/spring-projects/spring-data-build/blob/main/CONTRIBUTING.adoc so we should not add redundancy here.

@ricardohsmello ricardohsmello force-pushed the update-docs-readme-and-encryption branch from c3bfa5f to 3e3086d Compare June 5, 2025 15:26
Reconciled changes from main with the updated code sample that includes the required `keyId(...)` configuration for Queryable Encryption using the Range algorithm.

Signed-off-by: Ricardo Mello <[email protected]>
@ricardohsmello ricardohsmello force-pushed the update-docs-readme-and-encryption branch from d61c9fd to 60107a8 Compare June 5, 2025 15:33
@ricardohsmello ricardohsmello reopened this Jun 6, 2025
@ricardohsmello
Copy link
Author

Hi! All checks are now passing and the conflict has been resolved. Let me know if there's anything else I should adjust. Thanks again for the review! 😊

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: waiting-for-triage An issue we've not yet triaged
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants