-
Notifications
You must be signed in to change notification settings - Fork 1
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
Random channel names #131
base: master
Are you sure you want to change the base?
Random channel names #131
Conversation
…s for temporary channel name
@@ -1,5 +1,546 @@ | |||
# frozen_string_literal: true | |||
|
|||
adjective_list = [ |
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.
File temp_voice_event_container.rb
has 597 lines of code (exceeds 250 allowed). Consider refactoring.
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.
lolololol 😂
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.
ummm y codeclimate forget this complaint for later on???
One thing I was thinking about for this, it might be nice to have a setting that turns on/off the random names, versus the use of the user's name, just because in different instances, either would be helpful. |
Co-authored-by: Jordan Miller <[email protected]>
Co-authored-by: Jordan Miller <[email protected]>
Co-authored-by: Jordan Miller <[email protected]>
Code Climate has analyzed commit e76c5df and detected 0 issues on this pull request. View more on Code Climate. |
Just throwing up a PR to make things easier to look at/review.