-
Notifications
You must be signed in to change notification settings - Fork 12
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
User Actions #9
User Actions #9
Conversation
jeremierobert-coveo
commented
Jul 18, 2019
1228d8a
to
d0c9d1d
Compare
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.
:vraimentvraimentnice: !!
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.
Skipped last test files because I'd just repeat for the tenth time what I wrote before.
The tests are really not digest or reviewable. Try to use comments and spacing wisely to ensure they're reviewables
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.
Good job, looks nice !
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.
d52713d
to
81c95da
Compare
81c95da
to
fdaea8a
Compare
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.
Fix the last discussions I re-opened and it's approved.
a418e36
to
b58fb3b
Compare
b58fb3b
to
9f661e6
Compare