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

fix: create a logger instance per class instead of per instance #792

Merged
merged 1 commit into from
Jul 19, 2023

Conversation

lburgazzoli
Copy link
Contributor

No description provided.

@lburgazzoli lburgazzoli requested a review from a team July 19, 2023 09:03
@lburgazzoli lburgazzoli changed the title fix: create a logger instance per class instead of per class instance fix: create a logger instance per class instead of per instance Jul 19, 2023
@Delawen Delawen enabled auto-merge (rebase) July 19, 2023 09:05
@sonarcloud
Copy link

sonarcloud bot commented Jul 19, 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
2.3% 2.3% Duplication

@codecov
Copy link

codecov bot commented Jul 19, 2023

Codecov Report

Merging #792 (d3cb2d6) into main (acffcff) will not change coverage.
The diff coverage is 58.18%.

@@            Coverage Diff            @@
##               main     #792   +/-   ##
=========================================
  Coverage     79.24%   79.24%           
- Complexity      160      167    +7     
=========================================
  Files            56       56           
  Lines          3040     3040           
  Branches        529      529           
=========================================
  Hits           2409     2409           
  Misses          415      415           
  Partials        216      216           
Impacted Files Coverage Δ
...kend/api/service/deployment/DeploymentService.java 50.00% <14.28%> (ø)
...elet/KameletBindingDeploymentGeneratorService.java 73.63% <25.00%> (ø)
...tor/kamelet/KameletDeploymentGeneratorService.java 67.30% <25.00%> (ø)
.../backend/api/resource/v1/IntegrationsResource.java 72.22% <40.00%> (ø)
...kaoto/backend/metadata/parser/JarParseCatalog.java 73.33% <44.44%> (ø)
.../backend/api/resource/v1/CapabilitiesResource.java 66.66% <50.00%> (ø)
...ackend/api/resource/v1/ViewDefinitionResource.java 45.45% <50.00%> (ø)
...rser/step/camelroute/CamelRestDSLParseCatalog.java 93.75% <50.00%> (ø)
...data/parser/step/kamelet/KameletFileProcessor.java 83.13% <50.00%> (ø)
...elroute/IntegrationDeploymentGeneratorService.java 71.42% <50.00%> (ø)
... and 19 more

@Delawen Delawen merged commit 78f2cfc into KaotoIO:main Jul 19, 2023
@lburgazzoli lburgazzoli deleted the static-logger branch July 19, 2023 10:17
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.

2 participants