-
Notifications
You must be signed in to change notification settings - Fork 618
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
MINOR: Fix typos in org.apache.solr.core
package
#1629
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for attending to details like this!
Thanks for the reviews - please feel free to merge this at your earliest convenience, as it seems I don't have access to do so myself. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I read through the PR and have a few minor nits in the same spirit. I left the more stylistic editorials alone.
Thanks again for the PR!
solr/core/src/java/org/apache/solr/core/backup/repository/BackupRepositoryFactory.java
Outdated
Show resolved
Hide resolved
…upRepositoryFactory.java same capitalization
I pushed the changes up so I don't hold it up. @AndreyBozhko thanks for the contribution and I look forward to more in the future. |
@AndreyBozhko can you add an entry in |
This reverts commit 07533d6.
Thanks @MarcusSorealheis for the suggestion, I added a CHANGES entry. |
This should be backported to 9.3, no reason to keep it just for |
I'll back port next week. |
* solr core - fix typos * solr core - fix typos * solr core - fix typos * run ./gradlew tidy * Update solr/core/src/java/org/apache/solr/core/backup/repository/BackupRepositoryFactory.java same capitalization * a few typos. * add CHANGES entry * revert "add CHANGES entry" This reverts commit 07533d6. * add CHANGES entry --------- Co-authored-by: Marcus <marcuseagan@gmail.com> (cherry picked from commit 810783d)
backported to 9.4 |
Description
Fix a number of typos in
org.apache.solr.core
package - mostly in javadoc and comments.Checklist
Please review the following and check all that apply:
main
branch../gradlew check
.