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

WINDUPRULE-379 EAP 7.2 Migration: Hibernate ORM 5.1 to Hibernate ORM … #365

Merged
merged 10 commits into from Mar 6, 2019

Conversation

PhilipCattanach
Copy link
Contributor

…5.3 App Migration - references to org.hibernate.HibernateException

…5.3 App Migration - references to org.hibernate.HibernateException
Copy link
Member

@mrizzi mrizzi left a comment

Choose a reason for hiding this comment

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

Please remove the .gitignore file from this commit.
I suggested some other changes to the rule

rules-reviewed/eap7/eap71/hibernate51-53.windup.xml Outdated Show resolved Hide resolved
rules-reviewed/eap7/eap71/hibernate51-53.windup.xml Outdated Show resolved Hide resolved
rules-reviewed/eap7/eap71/hibernate51-53.windup.xml Outdated Show resolved Hide resolved
mrizzi and others added 6 commits March 6, 2019 09:09
Co-Authored-By: PhilipCattanach <31246010+PhilipCattanach@users.noreply.github.com>
Co-Authored-By: PhilipCattanach <31246010+PhilipCattanach@users.noreply.github.com>
Co-Authored-By: PhilipCattanach <31246010+PhilipCattanach@users.noreply.github.com>
Co-Authored-By: PhilipCattanach <31246010+PhilipCattanach@users.noreply.github.com>
@PhilipCattanach
Copy link
Contributor Author

Applied changes requested in the PR. The test.xml had to be changed to to ensure the issue count was correct. Also renamed the class name and constructor name in the .java file. Retested locally and the quotes in the message body serve to highlight the class and property names. Also the link takes the user to the Exception section of the document. Thanks Marco - All of these changes add value.

@mrizzi
Copy link
Member

mrizzi commented Mar 6, 2019

Once the builds will work, i'll merge this 👍

@mrizzi mrizzi merged commit 29d48de into windup:master Mar 6, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants