-
-
Notifications
You must be signed in to change notification settings - Fork 124
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
Style guide violation #572
Conversation
There is a style guide violation in line 166 and 167 so I change it according to style guide Signed-off-by: Mohit Bansal <78220157+MohitBansal321@users.noreply.github.com>
There is a style guide violation in line 54 so i change it according to style guide Signed-off-by: Mohit Bansal <78220157+MohitBansal321@users.noreply.github.com>
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.
Thanks so much for this contribution! I've made some notes for you to think about before this is merged. Let me know your thoughts.
In lines 166,167 and 169 Signed-off-by: Mohit Bansal <78220157+MohitBansal321@users.noreply.github.com>
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.
Couple of things here.
- I've given a suggestion of what I might do on this ticket. Take a look and have a think.
- This file actually has more lines referring to arr. Would you mind taking a look at those as well?
Signed-off-by: Mohit Bansal <78220157+MohitBansal321@users.noreply.github.com>
Signed-off-by: Mohit Bansal <78220157+MohitBansal321@users.noreply.github.com>
let me know all things are okay |
What does this change?
Module: js-core-1 , workshops
Week(s):week-4 , js-testing-workshop
#570 #569
Description
There is a style guide violation so I changed it according to style guide
Who needs to know about this?
@SallyMcGrath
Rendered Pages