Skip to content

Enhancement: Extract UuidProvider from StringProvider#230

Merged
localheinz merged 1 commit intomainfrom
feature/uuid-provider
Jun 27, 2023
Merged

Enhancement: Extract UuidProvider from StringProvider#230
localheinz merged 1 commit intomainfrom
feature/uuid-provider

Conversation

@localheinz
Copy link
Copy Markdown
Member

This pull request

  • extracts a UuidProvider from the StringProvider

@localheinz localheinz self-assigned this Jun 27, 2023
@localheinz localheinz requested a review from ergebnis-bot as a code owner June 27, 2023 11:22
@localheinz localheinz force-pushed the feature/uuid-provider branch from 244c7ec to 41114bb Compare June 27, 2023 11:22
@codecov
Copy link
Copy Markdown

codecov bot commented Jun 27, 2023

Codecov Report

Merging #230 (cf31067) into main (2935b01) will increase coverage by 0.17%.
The diff coverage is 100.00%.

@@             Coverage Diff              @@
##               main     #230      +/-   ##
============================================
+ Coverage     97.37%   97.55%   +0.17%     
- Complexity       46       50       +4     
============================================
  Files             9       10       +1     
  Lines           229      245      +16     
============================================
+ Hits            223      239      +16     
  Misses            6        6              
Impacted Files Coverage Δ
src/UuidProvider.php 100.00% <100.00%> (ø)

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

@localheinz localheinz force-pushed the feature/uuid-provider branch 3 times, most recently from b960e0c to 8dce2dc Compare June 27, 2023 11:28
@localheinz localheinz force-pushed the feature/uuid-provider branch from 8dce2dc to cf31067 Compare June 27, 2023 11:38
@localheinz localheinz merged commit 15a3db3 into main Jun 27, 2023
@localheinz localheinz deleted the feature/uuid-provider branch June 27, 2023 11:57
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.

2 participants