Skip to content

Conversation

@timneutkens
Copy link
Member

Solves a bunch of inconsistencies in handling React elements too.

@timneutkens timneutkens requested a review from dav-is January 25, 2019 15:31
Copy link
Contributor

@dav-is dav-is left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice 👍

@timneutkens timneutkens merged commit adfdc79 into vercel:canary Jan 25, 2019
@timneutkens timneutkens deleted the add/move-head-typescript branch January 25, 2019 15:43
"git add"
],
"*.tsx": [
"tslint -c tslint.json 'packages/**/*.ts' --fix",

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't this be "tslint -c tslint.json 'packages/**/*.tsx' --fix"?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@damianfrizzi actually it shouldn't have the pages/**/*.ts as lint-staged adds the file to be linted automatically to the command (so that it only runs on the changed file) could you create a PR? 💯

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There you go #6145

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Merged!

@lock lock bot locked as resolved and limited conversation to collaborators Jan 26, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants