Skip to content

Conversation

@bouwew
Copy link
Contributor

@bouwew bouwew commented Feb 3, 2024

No description provided.

@bouwew bouwew requested a review from CoMPaTech February 3, 2024 13:06
@codecov
Copy link

codecov bot commented Feb 3, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (7bc997e) 100.00% compared to head (0415ddc) 100.00%.

Additional details and impacted files
@@            Coverage Diff             @@
##              main      #510    +/-   ##
==========================================
  Coverage   100.00%   100.00%            
==========================================
  Files           11        19     +8     
  Lines         2669      3302   +633     
==========================================
+ Hits          2669      3302   +633     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@bouwew bouwew force-pushed the split_legacy_active branch 2 times, most recently from 9d2ff8f to df0e7e7 Compare February 3, 2024 13:16
Copy link
Member

@CoMPaTech CoMPaTech left a comment

Choose a reason for hiding this comment

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

Let's see where the tooling concurs on your commits, not reviewing this as per large refactor. We should merge and see what collateral damage emerges.

@CoMPaTech
Copy link
Member

Nice (and a lot of) work! Kudo's @bouwew

@bouwew bouwew force-pushed the split_legacy_active branch 2 times, most recently from a2530ef to 28bd71d Compare February 3, 2024 17:01
@bouwew bouwew force-pushed the split_legacy_active branch from 73a8452 to 9e7a8cb Compare February 4, 2024 09:36
@bouwew bouwew force-pushed the split_legacy_active branch from 86aa919 to 4fe2784 Compare February 4, 2024 13:13
@bouwew bouwew force-pushed the split_legacy_active branch from a25439b to ad42639 Compare February 4, 2024 14:29
@bouwew bouwew marked this pull request as ready for review February 4, 2024 14:38
@bouwew bouwew requested a review from a team as a code owner February 4, 2024 14:38
@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 4, 2024

Quality Gate Failed Quality Gate failed

Failed conditions

8.3% Duplication on New Code (required ≤ 3%)

See analysis details on SonarCloud

@bouwew bouwew merged commit 4802db8 into main Feb 4, 2024
@bouwew bouwew deleted the split_legacy_active branch February 4, 2024 17:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants