-
Notifications
You must be signed in to change notification settings - Fork 0
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
Upgrade eslint #52
Upgrade eslint #52
Conversation
0542536
to
a5ca765
Compare
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.
Lijkt allemaal te werken 👍
}) | ||
|
||
export default [ | ||
...compat.extends('plugin:@typescript-eslint/recommended'), |
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.
Eerst extenden we van @react-native
nu deze. Waarom?
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.
Voor zover ik het begrijp: extends bestaat niet meer sinds eslint 8.0.0. In plaats daarvan bestaat de configuratie uit een array van configuratie-objecten. Er is een utility FileCompat
om hieraan eslint-configuraties toe te voegen die nog geen flat eslint-configuration hebben. De helper fixupPluginRules
voor plugins doet iets dergelijks, is meer low level, en wordt intern door FileCompat
gebruikt.
De configuratie in deze PR werkt voor de eslint packages die wij gebruiken. Als alle eslint packages support voor eslint 8.0.0 hebben gekregen, dan hebben we FlatCompat
en fixupPluginRules
niet meer nodig.
"react": "18.2.0", | ||
"react-native": "0.74.3", | ||
"react-native-render-html": "^6.3.4", | ||
"react-test-renderer": "18.2.0", | ||
"ts-jest": "^29.2.2", | ||
"typescript": "5.0.4" | ||
"typescript": "5.0.4", | ||
"typescript-eslint": "^7.18.0" |
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.
Deze heeft inmiddels versie 8, maar ik weet nou niet of dit makkelijk is om te gebruiken
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.
Ten tijde van het maken van deze PR was 7.18.0 de meest recente versie. Daarmee is deze PR ook getest. Voor het bijwerken van dependencies hebben we een aparte user story.
@@ -1,11 +1,14 @@ | |||
{ | |||
"name": "@observation.org/react-native-components", | |||
"version": "1.42.0", | |||
"version": "1.43.0", |
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.
Weet niet of het echt nodig is, maar voor de zekerheid ook maar de version verhoogd.
Resolves: https://github.com/observation/app/issues/453