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

Adding Industry Segments Class #1148

Merged
merged 2 commits into from
Jul 26, 2018
Merged

Conversation

cdesch
Copy link
Contributor

@cdesch cdesch commented Feb 9, 2018

Adding Industry to options based off of the Industry Classification Benchmark

Adding Industry Segments to options based off of the Industry Classification Benchmark
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.

It seems that we have a few merge conflicts. Could you please rebase this branch, install the new dependencies, fix the rubocop violations and let me know when it's ready for review again please?

@vbrazo
Copy link
Member

vbrazo commented Jul 1, 2018

@cdesch I just fixed the merge issues and it seems that Travis is red because of Rubocop.

@vbrazo
Copy link
Member

vbrazo commented Jul 21, 2018

Any updates on this PR?

Copy link
Contributor Author

@cdesch cdesch left a comment

Choose a reason for hiding this comment

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

Looks Good

@cdesch
Copy link
Contributor Author

cdesch commented Jul 26, 2018

The changes look good. Thanks

@vbrazo vbrazo merged commit 8366e6d into faker-ruby:master Jul 26, 2018
davidmorton0 pushed a commit to davidmorton0/faker that referenced this pull request Jul 12, 2021
Adding Industry Segments to options based off of the Industry Classification Benchmark
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