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

Cleanup games #2155

Merged
merged 16 commits into from Oct 14, 2020
Merged

Cleanup games #2155

merged 16 commits into from Oct 14, 2020

Conversation

mathisto
Copy link
Contributor

Issue#

No-Story

Description:

Look ma! I'm gaming!
This PR cleans up the Games:: family's YAML content.

  • All files were run through rake reformat_yaml and hand-checked for breaking changes during the automation process.
  • Fixed a small oversight where a Pokemon move named Soul Reflect: Ankh (et al.), was being unintentionally interpreted by YAML as the syntax for defining a key-value pair.
  • As an added bonus, I augmented/completed the character section of the Games::Fallout generator using data scraped from the fandom Fallout wiki.

@mathisto
Copy link
Contributor Author

If it's not too much trouble, I'd like to request this PR receive the hacktoberfest-accepted tag. 🙇

@@ -0,0 +1,15 @@
# Faker::Fantasy::Tolkien
Copy link
Contributor

@Zeragamba Zeragamba Oct 13, 2020

Choose a reason for hiding this comment

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

slight bit of contamination from #2154. Can we clean this up a bit before merging?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ugh. Embarrassing! I'll def sweep up.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@Zeragamba The deed is done. I rebased and just dropped that commit.

lib/locales/en/myst.yml Show resolved Hide resolved
lib/locales/en/street_fighter.yml Show resolved Hide resolved
README.md Show resolved Hide resolved
README.md Show resolved Hide resolved
@Zeragamba Zeragamba merged commit fea49b8 into faker-ruby:master Oct 14, 2020
@Zeragamba
Copy link
Contributor

Thanks!

droznyk pushed a commit to droznyk/faker that referenced this pull request Oct 23, 2020
* Add minecraft to README and alphabetize

* Reformat dota yaml

* Reformat elder_scrolls yaml

* Reformat fallout yaml

* Compelete fallout character list

* Reformat half_life yaml

* Reformat heroes yaml

* Reformat heroes_of_the_storm yaml

* Reformat league of legends yaml

* Reformat myst yaml

* Reformat overwatch yaml

* Reformat pokemon yaml

* Fix soul reflect colon and whitespace

* Reformat withcer yaml

* Reformat zelda yaml

* Restore comments to myst. Standardize yaml.
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.

None yet

2 participants