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

feature(rest-dsl): Adding basic icon to REST DSL steps #681

Merged
merged 2 commits into from
May 31, 2023

Conversation

Delawen
Copy link
Collaborator

@Delawen Delawen commented May 30, 2023

Up till now, it was using the default "rest" string. Now it has a real icon.
rest-api

@Delawen Delawen requested a review from a team May 30, 2023 11:45
@@ -70,6 +74,55 @@ private static Step getVerbStep(String verb) {
return step;
}

@NotNull
Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Seriously? You are moved down a couple of lines because of new code and suddenly you are all brand new?

@sonarqubecloud
Copy link

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 2 Code Smells

No Coverage information No Coverage information
2.2% 2.2% Duplication

@codecov
Copy link

codecov bot commented May 30, 2023

Codecov Report

Merging #681 (2a99b4f) into main (4d3c1d5) will increase coverage by 0.31%.
The diff coverage is 95.34%.

@@             Coverage Diff              @@
##               main     #681      +/-   ##
============================================
+ Coverage     79.01%   79.32%   +0.31%     
- Complexity      148      149       +1     
============================================
  Files            48       48              
  Lines          2678     2685       +7     
  Branches        423      424       +1     
============================================
+ Hits           2116     2130      +14     
+ Misses          384      379       -5     
+ Partials        178      176       -2     
Impacted Files Coverage Δ
...rser/step/camelroute/CamelRestDSLParseCatalog.java 93.75% <95.34%> (-2.15%) ⬇️

... and 3 files with indirect coverage changes

@Delawen Delawen merged commit 0e619ba into kaoto-archive:main May 31, 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.

1 participant