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
Performance++ #97
Performance++ #97
Conversation
Is this PR still in progress? |
Not necessarily, you can merge it if you want |
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, just address the comments I made and I'll merge!
query = args.slice(1).join(' '); | ||
source = 'youtube'; | ||
const platform = args[0].toLowerCase(); | ||
query = args.slice(1).join(' '); |
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.
This only works if a platform and a query are provided. It will not work if only a query is provided.
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.
ahh I see it now, I will fix this in next commit
src/commands/settings/Profile.js
Outdated
if (!u.bio) bio = 'No bio set. To set your bio type `ear bio <desired bio>`'; | ||
else bio = u.bio; | ||
|
||
const bio = u ? u.bio : 'No bio set. To set your bio type `ear bio <desired bio>`'; |
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.
The if statement above already checks if there is a user. This should be changed to u.bio ? u.bio : '...';
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.
True, didn't think of that. I will change that to nullish coalescing operator
Some changes to clean things up a bit