Skip to content

Merges changes from commons-ognl and cleans up code#155

Merged
lukaszlenart merged 11 commits intomasterfrom
next-step
Sep 1, 2022
Merged

Merges changes from commons-ognl and cleans up code#155
lukaszlenart merged 11 commits intomasterfrom
next-step

Conversation

@lukaszlenart
Copy link
Copy Markdown
Collaborator

Commons OGNL supposed to supersede OGNL 3.x and be used instead of the 3.x branch, yet that never happened even if some preparations have been made, I took a time to merge some important fixes back into 3.x branch and fix some issues

@lukaszlenart
Copy link
Copy Markdown
Collaborator Author

@harawata @JCgH4164838Gh792C124B5 @quaff I'm working on a new version which can have breaking changes, for sure I want to integrate #140 into that with @harawata comments

@lukaszlenart
Copy link
Copy Markdown
Collaborator Author

@davoustp it should be interested for you as well

Commons OGNL supposed to supersede OGNL 3.x and be used instead of the 3.x branch, yet that never happened even if some preparations have been made, I took a time to merge some important fixes back into 3.x branch and fix some issues
@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud bot commented Sep 1, 2022

SonarCloud Quality Gate failed.    Quality Gate failed

Bug C 2 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 373 Code Smells

No Coverage information No Coverage information
0.8% 0.8% Duplication

@lukaszlenart lukaszlenart marked this pull request as ready for review September 1, 2022 11:54
@lukaszlenart lukaszlenart merged commit c6db7c8 into master Sep 1, 2022
@lukaszlenart lukaszlenart deleted the next-step branch September 1, 2022 12:00
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.

1 participant