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

Created New Method Faker::Quote.fortune_cookie #2112

Merged

Conversation

metalogica
Copy link
Contributor

Description:

When prototyping applications it's sometimes useful having a fun and quirky little messages to stand as placeholders for user content in the app. Fortune cookies are both profound and silly. I use something similar in my own projects, so I thought it could be something of use to others in the community too!

@@ -164,3 +164,4 @@ en:
"I want my damn cart back"
]
yoda: ["Use your feelings, Obi-Wan, and find him you will.", "Already know you that which you need.", "Adventure. Excitement. A Jedi craves not these things.", "At an end your rule is, and not short enough it was!", "Around the survivors a perimeter create.", "Soon will I rest, yes, forever sleep. Earned it I have. Twilight is upon me, soon night must fall.", "Not if anything to say about it I have", "Through the Force, things you will see. Other places. The future - the past. Old friends long gone.", "Ow, ow, OW! On my ear you are!", "The dark side clouds everything. Impossible to see the future is.", "Size matters not. Look at me. Judge me by my size, do you? Hmm? Hmm. And well you should not. For my ally is the Force, and a powerful ally it is. Life creates it, makes it grow. Its energy surrounds us and binds us. Luminous beings are we, not this crude matter. You must feel the Force around you; here, between you, me, the tree, the rock, everywhere, yes. Even between the land and the ship.", "Younglings, younglings gather ’round.", "Luminous beings are we - not this crude matter.", "Clear your mind must be, if you are to find the villains behind this plot.", "Always two there are, no more, no less. A master and an apprentice.", "Do. Or do not. There is no try.", "Much to learn you still have my old padawan. ... This is just the beginning!", "Good relations with the Wookiees, I have.", "Ready are you? What know you of ready? For eight hundred years have I trained Jedi. My own counsel will I keep on who is to be trained. A Jedi must have the deepest commitment, the most serious mind. This one a long time have I watched. All his life has he looked away - to the future, to the horizon. Never his mind on where he was. Hmm? What he was doing. Hmph. Adventure. Heh. Excitement. Heh. A Jedi craves not these things. You are reckless.", "Truly wonderful, the mind of a child is.", "Always pass on what you have learned.", "Once you start down the dark path, forever will it dominate your destiny, consume you it will.", "Mudhole? Slimy? My home this is!", "Yes, a Jedi’s strength flows from the Force. But beware of the dark side. Anger, fear, aggression; the dark side of the Force are they. Easily they flow, quick to join you in a fight. If once you start down the dark path, forever will it dominate your destiny, consume you it will, as it did Obi-Wan’s apprentice.", "Do not assume anything Obi-Wan. Clear your mind must be if you are to discover the real villains behind this plot.", "Death is a natural part of life. Rejoice for those around you who transform into the Force. Mourn them do not. Miss them do not. Attachment leads to jealously. The shadow of greed, that is.", "Like fire across the galaxy the Clone Wars spread. In league with the wicked Count Dooku, more and more planets slip. Against this threat, upon the Jedi Knights falls the duty to lead the newly formed army of the Republic. And as the heat of war grows, so, to, grows the prowess of one most gifted student of the Force.", "Hmm. In the end, cowards are those who follow the dark side.", "Strong is Vader. Mind what you have learned. Save you it can.", "Pain, suffering, death I feel. Something terrible has happened. Young Skywalker is in pain. Terrible pain", "Difficult to see. Always in motion is the future...", "You will find only what you bring in.", "Feel the force!", "Reckless he is. Matters are worse.", "That is why you fail.", "Your weapons, you will not need them.", "To answer power with power, the Jedi way this is not. In this war, a danger there is, of losing who we are."]
fortune_cookie: ["This cookie senses that you are superstitious; it is an inclination that is bad for your mental health.", "Goodness comes to those who wait.", "Two wrongs do not make a right.", "A rolling stone gathers no moss.", "Destiny is what you choose it to be; fate is what happens when you relinquish your freedom", "Failure is fuel for victory", "A Black Cat is not bad luck for a blind man.", "To hesitate is to be lost."]
Copy link
Contributor

Choose a reason for hiding this comment

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

To make it easier to find what was added or removed, we would prefer one item per line using the dash syntax.

If you want to update the others in this file that'd be awesome too.

lib/faker/quotes/quote.rb Outdated Show resolved Hide resolved
… reformatted arrays of strings to dash syntax in locales/en/faker/quote.yml
@metalogica
Copy link
Contributor Author

@Zeragamba @koic
I've pushed a new commit with the requested changes!

@Zeragamba
Copy link
Contributor

Looks like there's a conflict. Can you update from master?

@metalogica
Copy link
Contributor Author

@Zeragamba I've merged the master branch and all checks have passed!

Copy link
Member

@vbrazo vbrazo left a comment

Choose a reason for hiding this comment

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

Thanks 👍

@Zeragamba Zeragamba merged commit be0e822 into faker-ruby:master Sep 29, 2020
@Zeragamba
Copy link
Contributor

Thanks!

@metalogica
Copy link
Contributor Author

metalogica commented Oct 2, 2020

@Zeragamba , @vbrazo , @koic -- do you guys need any help in maintaining this project?
I'd love to do any technical or administrative work to help out! 💪

@Zeragamba
Copy link
Contributor

Opening PRs, issues, and doing code reviews are always welcome! Happy coding :)

droznyk pushed a commit to droznyk/faker that referenced this pull request Oct 23, 2020
* scaffolded fortune_cookie method, test & locale

* 1. changed Faker version to `next` in Faker::Quote.fortune_cookie; 2. reformatted arrays of strings to dash syntax in locales/en/faker/quote.yml
droznyk added a commit to droznyk/faker that referenced this pull request Oct 23, 2020
droznyk added a commit to droznyk/faker that referenced this pull request Oct 23, 2020
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