Skip to content
This repository was archived by the owner on Jan 4, 2022. It is now read-only.

Enhancement: Implement DataProvider\ResourceProvider#344

Merged
ergebnis-bot merged 1 commit intomainfrom
feature/resource-provider
Oct 5, 2020
Merged

Enhancement: Implement DataProvider\ResourceProvider#344
ergebnis-bot merged 1 commit intomainfrom
feature/resource-provider

Conversation

@localheinz
Copy link
Copy Markdown
Member

This PR

  • implements DataProvider\ResourceProvider

Follows #326.

@localheinz localheinz force-pushed the feature/resource-provider branch from a424f3e to 9dccc61 Compare October 5, 2020 12:14
@codecov
Copy link
Copy Markdown

codecov bot commented Oct 5, 2020

Codecov Report

Merging #344 into main will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff             @@
##                main      #344   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
- Complexity        84        85    +1     
===========================================
  Files             11        12    +1     
  Lines            285       288    +3     
===========================================
+ Hits             285       288    +3     
Impacted Files Coverage Δ Complexity Δ
src/DataProvider/ResourceProvider.php 100.00% <100.00%> (ø) 1.00 <1.00> (?)

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 2c263dc...9dccc61. Read the comment docs.

public static function resource(): \Generator
{
yield from self::provideDataForValues([
'resource' => \fopen(__FILE__, 'rb'),
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

This might result in fatal errors when too many files are open, but we can fix it when it does.

@ergebnis-bot ergebnis-bot self-requested a review October 5, 2020 12:15
@ergebnis-bot ergebnis-bot self-assigned this Oct 5, 2020
@ergebnis-bot ergebnis-bot merged commit 6dd7ff9 into main Oct 5, 2020
@ergebnis-bot ergebnis-bot deleted the feature/resource-provider branch October 5, 2020 12:15
@ergebnis-bot ergebnis-bot self-requested a review October 5, 2020 12:15
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants