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

Red Dead Redemption 2 Provider #766

Merged

Conversation

Majid-J
Copy link
Contributor

@Majid-J Majid-J commented Apr 14, 2023

Added a provider for Red Dead Redemption 2.

@kingthorin
Copy link
Collaborator

Could you remove the unrelated commits please

@codecov-commenter
Copy link

Codecov Report

Merging #766 (13351ee) into main (544c3f0) will increase coverage by 0.00%.
The diff coverage is 100.00%.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff            @@
##               main     #766   +/-   ##
=========================================
  Coverage     92.78%   92.78%           
- Complexity     2594     2615   +21     
=========================================
  Files           281      283    +2     
  Lines          5130     5173   +43     
  Branches        530      533    +3     
=========================================
+ Hits           4760     4800   +40     
- Misses          243      245    +2     
- Partials        127      128    +1     
Impacted Files Coverage Δ
.../main/java/net/datafaker/service/files/EnFile.java 93.33% <ø> (ø)
...afaker/providers/videogame/RedDeadRedemption2.java 100.00% <100.00%> (ø)
...afaker/providers/videogame/VideoGameProviders.java 100.00% <100.00%> (ø)

... and 4 files with indirect coverage changes

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

@Majid-J Majid-J force-pushed the feature/red-dead-redemption-2-provider branch from 13351ee to 4db27e7 Compare April 14, 2023 13:21
Copy link
Collaborator

@kingthorin kingthorin left a comment

Choose a reason for hiding this comment

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

LGTM

import net.datafaker.providers.base.AbstractProvider;

/**
* Red Dead Redemption 2 is an action-adventure game developed and published by Rockstar Games.
Copy link
Contributor

Choose a reason for hiding this comment

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

Thanks for adding this documentation here!!

@bodiam
Copy link
Contributor

bodiam commented Apr 14, 2023

Thanks for this great PR! Looks good to me too!

Comment on lines +13 to +21
TestSpec.of(redDeadRedemption2::protagonist, "red_dead_redemption2.protagonists"),
TestSpec.of(redDeadRedemption2::gangMember, "red_dead_redemption2.gang_members"),
TestSpec.of(redDeadRedemption2::majorCharacter, "red_dead_redemption2.major_characters"),
TestSpec.of(redDeadRedemption2::animal, "red_dead_redemption2.animals"),
TestSpec.of(redDeadRedemption2::state, "red_dead_redemption2.states"),
TestSpec.of(redDeadRedemption2::region, "red_dead_redemption2.regions"),
TestSpec.of(redDeadRedemption2::settlement, "red_dead_redemption2.settlements"),
TestSpec.of(redDeadRedemption2::quote, "red_dead_redemption2.quotes"),
TestSpec.of(redDeadRedemption2::weapon, "red_dead_redemption2.weapons")
Copy link
Collaborator

Choose a reason for hiding this comment

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

would be better if it satisfies abc order

Comment on lines +13 to +21
TestSpec.of(redDeadRedemption2::protagonist, "red_dead_redemption2.protagonists"),
TestSpec.of(redDeadRedemption2::gangMember, "red_dead_redemption2.gang_members"),
TestSpec.of(redDeadRedemption2::majorCharacter, "red_dead_redemption2.major_characters"),
TestSpec.of(redDeadRedemption2::animal, "red_dead_redemption2.animals"),
TestSpec.of(redDeadRedemption2::state, "red_dead_redemption2.states"),
TestSpec.of(redDeadRedemption2::region, "red_dead_redemption2.regions"),
TestSpec.of(redDeadRedemption2::settlement, "red_dead_redemption2.settlements"),
TestSpec.of(redDeadRedemption2::quote, "red_dead_redemption2.quotes"),
TestSpec.of(redDeadRedemption2::weapon, "red_dead_redemption2.weapons")
Copy link
Collaborator

Choose a reason for hiding this comment

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

would be better if it satisfies abc order

Copy link
Contributor

Choose a reason for hiding this comment

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

So picky :)

Copy link
Contributor

Choose a reason for hiding this comment

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

(And sorry, I missed your comment just when I was merging it!)

Copy link
Collaborator

Choose a reason for hiding this comment

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

probably that was the reason github has rejected with submission of my first comment...
ok,,

@bodiam bodiam merged commit 24ca17e into datafaker-net:main Apr 14, 2023
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.

5 participants