Skip to content

Update Dota2Provider dota2.yml#1435

Merged
kingthorin merged 1 commit intodatafaker-net:mainfrom
therepanic:update-dota2-game-provider
Nov 9, 2024
Merged

Update Dota2Provider dota2.yml#1435
kingthorin merged 1 commit intodatafaker-net:mainfrom
therepanic:update-dota2-game-provider

Conversation

@therepanic
Copy link
Copy Markdown
Contributor

Recently a new hero came out, I added him and his lines to our dota2.yml dataset

@therepanic therepanic force-pushed the update-dota2-game-provider branch from 8db0195 to 764c9fe Compare November 9, 2024 17:46
@codecov-commenter
Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.23%. Comparing base (ff110bf) to head (764c9fe).

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #1435      +/-   ##
============================================
+ Coverage     92.20%   92.23%   +0.03%     
- Complexity     3163     3164       +1     
============================================
  Files           320      320              
  Lines          6181     6181              
  Branches        593      593              
============================================
+ Hits           5699     5701       +2     
+ Misses          339      338       -1     
+ Partials        143      142       -1     

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

@kingthorin kingthorin merged commit 92427bd into datafaker-net:main Nov 9, 2024
@bodiam
Copy link
Copy Markdown
Contributor

bodiam commented Nov 10, 2024

@panic08 love it! Thank you for your contribution!

@therepanic therepanic deleted the update-dota2-game-provider branch March 3, 2025 14:29
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.

4 participants