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
Add minor changes to matching engine logic #24
Conversation
spawn("prettier", ["--write", sdkDir], { stdio: "inherit" }) | ||
.on("error", (err) => { | ||
console.error( | ||
"Failed to lint client files. Try installing prettier (`npm install --save-dev --save-exact prettier`)" |
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.
add comment that we should add a prettier config here (otherwise, will default to whatever your local prettier has rather than being consistent)
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 call, if you have one, can you quickly paste to this PR. Not opinionated here
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.
just leave the comment for now and we can add one in if a future diff comes in that looks ugly. As long as the default includes semicolons and double-quotes since that's what all of our editors currently do
Changes: