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

[fix:httpProcess] Remove unused configurations from plugins. #4882

Merged
merged 3 commits into from
Jul 20, 2023

Conversation

lahmXu
Copy link
Member

@lahmXu lahmXu commented Jul 19, 2023

Remove the unused configurations of the Request, paramMapping, and modifyResponse plugin, because this configuration is not used in the corresponding plugin code.

Make sure that:

  • You have read the contribution guidelines.
  • You submit test cases (unit or integration tests) that back your changes.
  • Your local test passed ./mvnw clean install -Dmaven.javadoc.skip=true.

@yu199195 yu199195 added this to the 2.6.0 milestone Jul 20, 2023
@codecov-commenter
Copy link

codecov-commenter commented Jul 20, 2023

Codecov Report

Merging #4882 (f0d6ff4) into master (9314ac1) will decrease coverage by 0.02%.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             master    #4882      +/-   ##
============================================
- Coverage     64.48%   64.47%   -0.02%     
+ Complexity     8244     8243       -1     
============================================
  Files          1166     1166              
  Lines         34232    34232              
  Branches       3067     3067              
============================================
- Hits          22076    22070       -6     
- Misses        10397    10405       +8     
+ Partials       1759     1757       -2     

see 3 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@yu199195 yu199195 merged commit 1a45cc3 into apache:master Jul 20, 2023
35 checks passed
@lahmXu lahmXu deleted the fix/httpProcess branch July 20, 2023 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants