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::Cosmere #1115

Merged
merged 2 commits into from
Jul 12, 2018
Merged

Add Faker::Cosmere #1115

merged 2 commits into from
Jul 12, 2018

Conversation

JauntyJames
Copy link
Contributor

I added a new class of terms from Brandon Sanderson's Cosmere books - I hope you like it! It's all tested and ready to go.

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.

Since it's been a long time that you opened this PR and we have integrated rubocop recently to the Faker project, I think it'd be great if you could update your branch with master and see if there are rubocop violations.

@vbrazo
Copy link
Member

vbrazo commented Jul 1, 2018

@JauntyJames I just updated your branch and it looks like we don't have rubocop violations.

@vbrazo vbrazo merged commit 8568945 into faker-ruby:master Jul 12, 2018
davidmorton0 pushed a commit to davidmorton0/faker that referenced this pull request Jul 12, 2021
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.

2 participants