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

Tcp plugin #4607

Merged
merged 30 commits into from
May 4, 2023
Merged

Tcp plugin #4607

merged 30 commits into from
May 4, 2023

Conversation

ywj1352
Copy link
Contributor

@ywj1352 ywj1352 commented May 3, 2023

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.

@codecov-commenter
Copy link

codecov-commenter commented May 3, 2023

Codecov Report

Merging #4607 (d09f20f) into master (a6c7831) will decrease coverage by 0.06%.
The diff coverage is 3.57%.

@@             Coverage Diff              @@
##             master    #4607      +/-   ##
============================================
- Coverage     67.72%   67.66%   -0.06%     
  Complexity     7833     7833              
============================================
  Files          1065     1067       +2     
  Lines         30578    30606      +28     
  Branches       2785     2785              
============================================
+ Hits          20708    20709       +1     
- Misses         8249     8276      +27     
  Partials       1621     1621              
Impacted Files Coverage Δ
...rg/apache/shenyu/common/dto/ProxySelectorData.java 0.00% <0.00%> (ø)
...common/dto/convert/selector/DiscoveryUpstream.java 0.00% <0.00%> (ø)
...ava/org/apache/shenyu/common/enums/PluginEnum.java 100.00% <100.00%> (ø)

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

@yu199195 yu199195 added this to the 2.6.0 milestone May 4, 2023
@yu199195 yu199195 merged commit 86a4c1c into apache:master May 4, 2023
35 checks passed
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.

None yet

3 participants