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

Properties case style fix #12559

Merged
merged 4 commits into from
Sep 26, 2020
Merged

Conversation

isaul32
Copy link
Contributor

@isaul32 isaul32 commented Sep 19, 2020


Please make sure the below checklist is followed for Pull Requests.

When you are still working on the PR, consider converting it to Draft (bellow reviewers) and adding skip-ci label, you can still see CI build result at your branch.

@isaul32 isaul32 changed the title Properties case fix Properties case style fix Sep 19, 2020
mapstruct_version=1.3.1.Final
archunit_junit5_version=0.14.1
<%_ if (searchEngine === 'elasticsearch') { _%>
log4j2_mock_version=0.0.2
Copy link
Contributor

Choose a reason for hiding this comment

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

👍 I forgot to remove it during the Elasticsearch upgrade: #11683
Could you do me a favour, and also remove it in pom.xml.ejs template?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sure 🙂

@isaul32 isaul32 marked this pull request as ready for review September 19, 2020 20:33
@isaul32 isaul32 force-pushed the properties-case-fix branch from 962b7ef to 12417f6 Compare September 20, 2020 05:10
@isaul32 isaul32 force-pushed the properties-case-fix branch from d79bdb5 to bf40eef Compare September 20, 2020 05:39
Copy link
Member

@DanielFran DanielFran left a comment

Choose a reason for hiding this comment

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

LGTM

@pascalgrimaud pascalgrimaud merged commit 205e0d1 into jhipster:master Sep 26, 2020
@pascalgrimaud
Copy link
Member

Thanks and congrats for your 1st contribution @isaul32 !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants