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

Add Faker::Movies::Departed Class #1931

Conversation

jaebradley
Copy link
Contributor

Summary

Adds the ability to select actors, characters, or quotes from the movie, The Departed.

The quotes were copied from IMDB.

The actors and characters were copied from IMDB.

When character names were incomplete on the IMDB website, they were corroborated against using The Departed Wiki.

Issue#

No-Story

Description

I envision using the class's character and actor names to populate the names of various test factories as well as using the quotes to populate text fields.

Adds the ability to select actors, characters, or quotes from the movie,
The Departed.
@@ -0,0 +1,11 @@
# Faker::Movies::Departed

Available since version next.
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Feel free to adjust this version value - I might have missed how to generate the next version (I did peek at the contribution guide, but could have misread the section on document generation).

Copy link
Contributor

Choose a reason for hiding this comment

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

For these docs, I don't think we keep track of when a set of generators were added.

@jaebradley
Copy link
Contributor Author

jaebradley commented Apr 8, 2020

@Zeragamba and @vbrazo are you folks the right people to tag for review? If not, please let me know so I can request review from those people.

Thanks!

@Zeragamba Zeragamba merged commit 53341f8 into faker-ruby:master May 23, 2020
@Zeragamba
Copy link
Contributor

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.

4 participants