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.

Sign up for GitHub

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

Jump to bottom

Eslint prettier #16

Open

mikaalnaik wants to merge 3 commits into main from eslint-prettier

Owner

mikaalnaik commented Mar 21, 2022 •

edited

Loading

Howdy fellow engineer!

🔗 🎫 Link to Issue If Available

🔨 The code change improves the stability or experience of the product.
😍 The implementation & user experience makes sense to you and us.
🪲 Debugged by the engineer.
🧪 Includes automated tests for the code that’s being changed.

Description

Please describe your pull request.e

⚖️ Notes for QA

What would be helpful for the person doing a quality assurance check on this PR?

💪 Thanks for helping build a better beach app for Torontonians! Keep on hacking!

mikaalnaik added 3 commits

March 21, 2022 08:32


          Refactor to modern JS

fed6284


          Add eslint

02672e6


          Add linting to build step

c271436

Owner Author

mikaalnaik commented Oct 31, 2022

need to resolve conflicts

Owner Author

mikaalnaik commented Oct 31, 2022

tests

uncode-review bot commented Nov 2, 2022

HOWDY FOLKS

uncode-review bot commented Nov 4, 2022

Est Review Time NaN
Complexity: 1

uncode-review bot commented Nov 4, 2022 •

edited

Loading

Estimates

Review Time NaN
Complexity: 🟢 1

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet