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.

Sign up for GitHub

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

Jump to bottom

[RHDM-1376] org.kie.dmn.validation fails on master with ibm jdk #2951

Merged
merged 2 commits into from
Jun 18, 2020
Merged

[RHDM-1376] org.kie.dmn.validation fails on master with ibm jdk #2951

merged 2 commits into from
Jun 18, 2020

Conversation

tkobayas
Copy link
Contributor

@tkobayas tkobayas commented Jun 17, 2020 •
edited
Loading

I was comparing QName to String before. Coercion happened to help in case of other JDKs but fixed now.

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

warning The version of Java (1.8.0_202) you have used to run this analysis is deprecated and we will stop accepting it from October 2020. Please update to at least Java 11.
Read more here

@tkobayas tkobayas marked this pull request as ready for review June 18, 2020 09:05
@tkobayas tkobayas requested review from tarilabs and hellowdan June 18, 2020 09:06
tarilabs
tarilabs approved these changes Jun 18, 2020
View reviewed changes
Copy link
Member

@tarilabs tarilabs left a comment

Choose a reason for hiding this comment

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

Thank you!

@mariofusco mariofusco merged commit fb58784 into apache:master Jun 18, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tarilabs tarilabs tarilabs approved these changes

@hellowdan hellowdan hellowdan approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@tkobayas @tarilabs @hellowdan @mariofusco