Skip to content
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

Update dependencies #53

Merged
merged 2 commits into from
Aug 14, 2024
Merged

Conversation

SjaakSchilperoort
Copy link
Member

@SjaakSchilperoort SjaakSchilperoort commented Aug 8, 2024

Contributes to: https://github.com/observation/app/issues/484

Also: fix for lint ignore that was accidentally removed

package.json Outdated
"react-native-render-html": "^6.3.4",
"react-test-renderer": "18.2.0",
"ts-jest": "^29.2.2",
"react-test-renderer": "18.3.1",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Volgens npx @rnx-kit/align-deps --requirements [email protected]:

 - react-test-renderer "18.3.1" should be "18.2.0"

@SjaakSchilperoort
Copy link
Member Author

@jonathan-waarneming-nl Deze branch geeft een issue in de Observation app. Als ik daar deze branch in package.json aangeef, dan verschijnt na het draaien van yarn de directory node_modules/@observation.org/react-native-components/node_modules. En daar valt jest over. Als je de develop branch van de deze repo gebruikt krijg je die node_modules directory niet. Heb jij enig idee waar dit verschil vandaan komt?

@jonathan-waarneming-nl
Copy link
Collaborator

Volgens mij komt dat door de verschillende versies van @FortAwesome packages. Als er verschillen zijn zet ie de versies die nodig zijn in de node_modules/@observation.ogr/react-native-components/node_modules. Jest probeert ze te transpilen, maar die worden vervolgens door jest gezien als fout, omdat de syntax niet klopt (typescript in een javascript file).

Als de versies hetzelfde zijn is 't probleem weg.

Je zou de dependencies @fortawesome/fontawesome-svg-core @fortawesome/pro-light-svg-icons @fortawesome/pro-solid-svg-icons @fortawesome/react-native-fontawesome uit develop van de app kunnen halen.

Of deze branch gebruiken in https://github.com/observation/app/pull/499 en mergen, dan heb worden steeds dezelfde versies gebruikt.

Volgens mij kun je jest ook instellen die de node_modules van dit packages niet worden getranspiled.

@SjaakSchilperoort
Copy link
Member Author

Okay, thanks, ik heb in de overeenkomstige PR voor de observation app de font awesome dependencies verwijderd.

Copy link
Collaborator

@jonathan-waarneming-nl jonathan-waarneming-nl left a comment

Choose a reason for hiding this comment

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

👍

@SjaakSchilperoort SjaakSchilperoort merged commit 565fee1 into develop Aug 14, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants