Skip to content

feat: add Tormenta RPG provider#1802

Open
lvspa wants to merge 5 commits intodatafaker-net:mainfrom
lvspa:feat/tormenta-provider
Open

feat: add Tormenta RPG provider#1802
lvspa wants to merge 5 commits intodatafaker-net:mainfrom
lvspa:feat/tormenta-provider

Conversation

@lvspa
Copy link
Copy Markdown

@lvspa lvspa commented Apr 24, 2026

Added the Tormenta RPG provider (a famous Brazilian RPG heavily inspired by D&D). Includes bestiary, character names, and cities. Follows the Dungeons & Dragons standard.

@what-the-diff
Copy link
Copy Markdown

what-the-diff Bot commented Apr 24, 2026

PR Summary

  • Introduction of Tormenta provider
    This change includes a new functional class called Tormenta. It's made exclusively to assist with tasks specific to the Tormenta RPG, such as accessing information about certain creatures, names, and cities.

  • Modification to BaseProviders
    The BaseProviders has been updated with a method called tormenta(). This offers an easy way to access an instance of Tormenta, thus enhancing the usefulness of the BaseProviders.

  • Creation of Localization file
    We've added a new localization file named 'tormenta.yml'. It stores collections of creature types, names, and cities related to the Tormenta RPG, making it easier for users to access and utilize this information.

  • New Tests for Tormenta
    To ensure the smooth operation of the Tormenta provider, a relevant test class named TormentaTest.java was created. It helps verify that the Tormenta provider is delivering the expected results consistently.

@asolntsev asolntsev added this to the 2.6.0 milestone Apr 25, 2026
@asolntsev asolntsev self-requested a review April 25, 2026 13:16
@asolntsev
Copy link
Copy Markdown
Collaborator

@lvspa Are these names only in English? Maybe you want to add the same names in Portuguese too?

@asolntsev
Copy link
Copy Markdown
Collaborator

compilation error:

13:16:14.047 [ERROR] /home/runner/work/datafaker/datafaker/src/main/java/net/datafaker/providers/base/BaseProviders.java:[3,83] package com.project.data.faker.datafaker.src.main.java.net.datafaker.providers.base does not exist

@lvspa
Copy link
Copy Markdown
Author

lvspa commented Apr 25, 2026

Hi @asolntsev !

Thank you for the feedback!

Regarding the translation, these are proper names specifically created for the Tormenta RPG setting. Since they are fictional proper nouns (fantasy names), they don't have a direct translation or a different version in Portuguese, they are used as-is in both languages.

I also apologize for the directory structure issues in the previous commits. I had some local environment and IDE configuration issues that caused the incorrect package paths. I have now performed a clean rebase, corrected the directory hierarchy to net.datafaker.providers.base, and ensured everything follows the project standard.

I'm really enjoying contributing to Datafaker and I plan to submit more PRs soon, specifically focusing on data that would be very valuable for Brazilian professional contexts, such as Brazilian phone numbers (landlines and mobile) and other localized formats.

Thanks for your patience and for maintaining this great project! 👍🏼

Copy link
Copy Markdown
Collaborator

@asolntsev asolntsev left a comment

Choose a reason for hiding this comment

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

Now this PR has 418 changed files.
Apparently, the folder "out" shouldn't be committed. @lvspa

@lvspa
Copy link
Copy Markdown
Author

lvspa commented Apr 25, 2026

I apologize for that; it was a silly mistake that went unnoticed. I'm still adjusting to the new development environment I'm using. Anyway, the error has been corrected. I hope everything works out now; I'm really excited to get it up and running :)

@kingthorin
Copy link
Copy Markdown
Collaborator

Have a look at the "Files changed" tab.

@lvspa
Copy link
Copy Markdown
Author

lvspa commented Apr 25, 2026

I believe it's finally clean now @kingthorin, thks!

I must apologize for the back-and-forth and the extra noise in this PR. I'm still getting used to how IntelliJ IDEA handles local artifacts and project structures, and I didn't realize it was 'auto-including' so much junk (like the out folder and local test files) into my commits.

It was a tough battle against the IDE, but I’ve now manually scrubbed the branch. I’ve removed the Main.java test file, cleared the build artifacts, and ensured that only the relevant files for the Tormenta provider are included in the 'Files changed' tab.

Thank you so much for your patience.

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.

3 participants