Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fixed problem with driver name. #44

Merged
merged 10 commits into from Jan 10, 2022
Merged

Fixed problem with driver name. #44

merged 10 commits into from Jan 10, 2022

Conversation

butschster
Copy link
Contributor

Covers Driver with unit tests

@roxblnfk roxblnfk added the type:bug Bug label Jan 10, 2022
@roxblnfk roxblnfk added this to the 2.1.x milestone Jan 10, 2022
@codecov
Copy link

codecov bot commented Jan 10, 2022

Codecov Report

Merging #44 (58720b5) into 2.x (4c15f4b) will decrease coverage by 0.02%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##                2.x      #44      +/-   ##
============================================
- Coverage     92.71%   92.69%   -0.03%     
- Complexity     1523     1551      +28     
============================================
  Files            93       93              
  Lines          3956     4024      +68     
============================================
+ Hits           3668     3730      +62     
- Misses          288      294       +6     
Impacted Files Coverage Δ
src/Driver/Driver.php 85.37% <100.00%> (-0.75%) ⬇️
src/Query/SelectQuery.php 94.85% <0.00%> (-0.60%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4c15f4b...58720b5. Read the comment docs.

@scrutinizer-notifier
Copy link

The inspection completed: 1 updated code elements

@roxblnfk roxblnfk merged commit 0e38747 into 2.x Jan 10, 2022
@roxblnfk roxblnfk deleted the feature/driver-name branch January 10, 2022 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

None yet

4 participants