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

Migrate to typescript with node v18 #1275

Conversation

ivangabriele
Copy link
Contributor

@ivangabriele ivangabriele commented Jul 20, 2022

[Description]

Linked issues

...

Linked PRs


  • Tests E2E (Cypress)

@ivangabriele ivangabriele force-pushed the ivan/build-frontend-migrate-to-typescript-with-node-18 branch 5 times, most recently from 8375e2a to a3dff97 Compare July 22, 2022 11:05
@ivangabriele ivangabriele force-pushed the ivan/build-frontend-migrate-to-typescript-with-node-18 branch 24 times, most recently from 255dbbf to 10b4ff6 Compare July 26, 2022 23:24
@ivangabriele ivangabriele force-pushed the ivan/build-frontend-migrate-to-typescript-with-node-18 branch from 1fedae7 to 608330d Compare August 2, 2022 07:47
@ivangabriele ivangabriele force-pushed the ivan/build-frontend-migrate-to-typescript-with-node-18 branch from 1ca06d8 to 5efba77 Compare August 2, 2022 09:02
@ivangabriele ivangabriele force-pushed the ivan/build-frontend-migrate-to-typescript-with-node-18 branch from 5efba77 to 6041a1e Compare August 2, 2022 09:23
@sonarqubecloud
Copy link

sonarqubecloud bot commented Aug 9, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 4 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

.github/workflows/cicd.yml Show resolved Hide resolved
frontend/.eslintrc Show resolved Hide resolved
frontend/.eslintrc Show resolved Hide resolved
infra/docker/DockerfileBuildApp Show resolved Hide resolved
docker network inspect monitorfish_network >/dev/null 2>&1 || docker network create monitorfish_network
docker compose -f ./infra/dev/docker-compose.yml up -d db
docker compose -f ./infra/dev/docker-compose.yml up flyway
Copy link
Collaborator

Choose a reason for hiding this comment

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

Chez moi, flyway se lance alors que la db n'est pas prête, ce qui fait planter flyway. Flyway fonctionne correctement si lancé une fois que la db est up.
Je n'ai pas pu aller au bout car l'image de l'app ne fonctionne pas sur mon m1.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Alors je veux bien qu'on regarde ça ensemble car pour le coup je suis la méthode officielle recommandée (et d'ailleurs utilisée par la team Github) pour faire des healthy checks sur Postgre. Mais il y a peut-être un soucis de compatibilité d'architecture sur l'image de base.

"test": "jest --config=./config/jest.config.js --detectOpenHandles",
"cypress:open": "cypress open --browser firefox --e2e",
"cypress:open:electron": "cypress open --browser electron --e2e",
"cypress:run": "cypress run --browser firefox --e2e",
Copy link
Collaborator

Choose a reason for hiding this comment

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

Je n'arrive pas à lancer les tests en local : Unknown file extension ".ts" for /Users/thomasbrosset/monitorfish/frontend/cypress.config.ts

Copy link
Contributor Author

@ivangabriele ivangabriele Aug 21, 2022

Choose a reason for hiding this comment

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

Ca pareil, il faudra qu'on regarde ensemble ! Je l'ai testé sur un Mac et un PC Ubuntu sans soucis de mon côté.

@louptheron louptheron closed this Aug 22, 2022
@louptheron louptheron deleted the ivan/build-frontend-migrate-to-typescript-with-node-18 branch August 22, 2022 08:56
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.

3 participants