-
Notifications
You must be signed in to change notification settings - Fork 76
Change findById(…) assert message to reflect the actual operation #345
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
Conversation
AbassAdeyemi
commented
Jan 12, 2021
•
edited
Loading
edited
- 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).
@AbassAdeyemi Please sign the Contributor License Agreement! Click here to manually synchronize the status of this Pull Request. See the FAQ for frequently asked questions. |
@pivotal-issuemaster This is an Obvious Fix |
@AbassAdeyemi This Pull Request contains an obvious fix. Signing the Contributor License Agreement is not necessary. |
Thank you very much for the clarification. Fortunately, I was able to also
figure that out early. Please, let me know if there are further actions
required of me for the changes to be eligible for merging.
…On Tue, Jan 12, 2021, 2:49 PM Pivotal Issuemaster ***@***.***> wrote:
@AbassAdeyemi <https://github.com/AbassAdeyemi> This Pull Request
contains an obvious fix. Signing the Contributor License Agreement is not
necessary.
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#345 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AJD5QSTL7FCHV7A3KBNMZ3TSZRHNLANCNFSM4V66QGRQ>
.
|
Thanks a lot. No further action is required, I need to merge this PR tomorrow or so. |