-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
Conversation
Adding Industry Segments to options based off of the Industry Classification Benchmark
There was a problem hiding this 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?
a359def
to
a5d7731
Compare
@cdesch I just fixed the merge issues and it seems that Travis is red because of Rubocop. |
Any updates on this PR? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks Good
The changes look good. Thanks |
Adding Industry Segments to options based off of the Industry Classification Benchmark
Adding
Industry
to options based off of the Industry Classification Benchmark