Skip to content
This repository has been archived by the owner on Nov 27, 2023. It is now read-only.

fix: Aggregate EIP also contains steps #825

Merged
merged 1 commit into from
Aug 4, 2023

Conversation

Delawen
Copy link
Collaborator

@Delawen Delawen commented Aug 3, 2023

Fixes #808

@Delawen Delawen requested a review from a team August 3, 2023 11:30
@sonarqubecloud
Copy link

sonarqubecloud bot commented Aug 3, 2023

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

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

No Coverage information No Coverage information
1.7% 1.7% Duplication

@codecov
Copy link

codecov bot commented Aug 3, 2023

Codecov Report

Merging #825 (1ad630b) into main (656ab1f) will increase coverage by 0.12%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##               main     #825      +/-   ##
============================================
+ Coverage     69.96%   70.08%   +0.12%     
- Complexity      199      200       +1     
============================================
  Files            69       69              
  Lines          3283     3283              
  Branches        555      555              
============================================
+ Hits           2297     2301       +4     
+ Misses          771      768       -3     
+ Partials        215      214       -1     
Files Changed Coverage Δ
...in/java/io/kaoto/backend/camel/KamelPopulator.java 87.59% <100.00%> (ø)

... and 2 files with indirect coverage changes

@lordrip
Copy link
Contributor

lordrip commented Aug 3, 2023

image

@Delawen Delawen merged commit 4be7f6e into kaoto-archive:main Aug 4, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] Aggregate EIP is not allowing nested steps
2 participants