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

Spongebob TV show collections #2619

Merged
merged 5 commits into from
Nov 15, 2022
Merged

Spongebob TV show collections #2619

merged 5 commits into from
Nov 15, 2022

Conversation

kleinjm
Copy link
Contributor

@kleinjm kleinjm commented Nov 8, 2022

Summary

Adds Spongebob TV show collections of characters, quotes, and episodes

image

Copy link

@holtbp holtbp left a comment

Choose a reason for hiding this comment

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

🌊
🍍

doc/tv_shows/spongebob.md Outdated Show resolved Hide resolved
Copy link
Contributor

@thdaraujo thdaraujo left a comment

Choose a reason for hiding this comment

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

haha this is great, thanks for the PR!

I left a few suggestions.

lib/locales/en/spongebob.yml Outdated Show resolved Hide resolved
lib/locales/en/spongebob.yml Outdated Show resolved Hide resolved
@kleinjm
Copy link
Contributor Author

kleinjm commented Nov 14, 2022

@thdaraujo I addressed the comments and reran tests & linting locally. Should be good to go!

On a side note, I've seen your name on the patch.io repo quite a bit from the early days. Nice to finally meet you!

@thdaraujo
Copy link
Contributor

@thdaraujo I addressed the comments and reran tests & linting locally. Should be good to go!

On a side note, I've seen your name on the patch.io repo quite a bit from the early days. Nice to finally meet you!

nice to meet you too, James! Hope my code didn't cause you too much trouble 😆

Copy link
Contributor

@thdaraujo thdaraujo left a comment

Choose a reason for hiding this comment

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

:shipit:

@thdaraujo thdaraujo merged commit d8b50a4 into faker-ruby:main Nov 15, 2022
@kleinjm
Copy link
Contributor Author

kleinjm commented Nov 29, 2022

@thdaraujo heads up that this change didn't make it in the 3.0.0 release but the readme on main has the docs

image

@thdaraujo
Copy link
Contributor

hey @kleinjm that's expected. We merged your changes after the v3 release, but it will be included in the next one. 😊 Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants