-
Notifications
You must be signed in to change notification settings - Fork 26.8k
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
Fix tsx workaround, bootstrap next.d.ts, add tslint deps #2885
Conversation
"jsRules": {}, | ||
"rules": { | ||
"quotemark": [true, "single", "jsx-double"], | ||
"semicolon": [true, "never"] |
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.
I've changed default rules, to match next.js coding style (but I don't like it). Is it possible to stick to default rules (using double quote and always append semicolons).
"@types/react": "^15.0.1", | ||
"concurrently": "^3.1.0", | ||
"typescript": "^2.1.5" | ||
"@types/node": "8.0.26", |
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.
This definitions are required by next.d.ts (and url import)
@timneutkens do you want me to rebase my PR, otherwise just close it, no problem |
@brikou looks okay, just rebase and ping me again, I'll merge 👌 |
@timneutkens rebased 👍 |
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.
Thanks!
@timneutkens thank you too! |
The primary goal of this PR is to solve issue explain in README by setting jsx to react-native.
Another big idea is to bootstrap a next.d.ts definition file as a reference (quite empty for now) but this is a bootstrap.
I also revamp the example to use next/link and so emphasize that there is no more problem :)