Closed Atremblay9 closed 4 weeks ago
This looks good and works for now but there are a few issues that @Atremblay9 has brought up in an in-person conversation.
I'm going to merge because it looks good to me (LGTM).
@allcontributors add @Atremblay9 for code
@arashsheyda
I've put up a pull request to add @Atremblay9! :tada:
What issue is this referencing?
393
Here is a screenshot!
Do these code changes work locally and have you tested that they fix the issue yourself?
Does the following command run without warnings or errors?
npm run pr-checks
Have you taken a look at our contributing guidelines?
My node version matches the one suggested when running
nvm use
?