openrewrite / openrewrite/rewrite
NameCaseConvention.LOWER_CAMEL should handle mixed case more properly
Open
Nobody has claimed this yet.
enhancement
- Dominant language
- Java
- Stars
- 3.7k
- Forks
- 570
- Avg merge
- 13h 12m
- Merged PRs (30d)
- 261
Description
Actual:
- PdfDocumentViewModel PDFviewModel = new PdfDocumentViewModel(mockPDF);
+ PdfDocumentViewModel pDFviewModel = new PdfDocumentViewModel(mockPDF);
Expected
- PdfDocumentViewModel PDFviewModel = new PdfDocumentViewModel(mockPDF);
+ PdfDocumentViewModel pdfViewModel = new PdfDocumentViewModel(mockPdf);
Update mockPdf instead of mockPDF.
(This was originally reported at https://github.com/openrewrite/rewrite-static-analysis/issues/12#issuecomment-1588208014)
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating the implementation and tests for NameCaseConvention.LOWER_CAMEL. Reproduce the mixed-case example from the issue and compare the actual pDFviewModel/mockPDF result with the expected pdfViewModel/mockPdf result. Done means the convention handles this case as shown and the relevant tests pass.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- devtools
- Issue type
- Bug
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100