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

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

Merged
merged 11 commits into from
Sep 1, 2022
Merged

Conversation

lukaszlenart
Copy link
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
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
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

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