GitHub actions #11
GitHub actions #11
Conversation
src/components/home/BookCards.tsx
Outdated
| @@ -1,4 +1,4 @@ | |||
| import { Card, CardActionArea, Stack, Typography } from "@mui/material" | |||
| import { Card, Stack, Typography } from "@mui/material" | |||
There was a problem hiding this comment.
El CI trata a los warnings como errores, asi que no empaqueta si tenemos warnings. Esto se puede cambiar si resulta engorroso a futuro, por ahora me parece apropiado para evitar problemillas que se nos pasen en local.
Este archivo fue modificado para sacar los warnings que no dejaban empaquetar en el release.
There was a problem hiding this comment.
Arrancar siendo estrictos está perfecto. Siempre se puede decidir dejar de serlo.
| - run: npm install | ||
|
|
||
| - name: Run tests | ||
| run: xvfb-run -a npm run test |
There was a problem hiding this comment.
La action GabrielBB/xvfb-action que estamos usando en Pilas-bloques, además de tener el último commit en noviembre del 2021 (no está siendo mantenida), probablemente deje de funcionar en invierno de este año: GabrielBB/xvfb-action#30. Una solución que dieron en ese mismo issue es usar xvfb-run que viene en la imagen de ubuntu-latest, por lo que no sería necesaria la action.
Si esto les parece, habría que cambiaron en el otro proyecto también
There was a problem hiding this comment.
🚀
VAMO!!!
Guarda que no subió el .dmg, fijate:
https://github.com/Program-AR/pilas-bloques-react/releases
Acá el error:
https://github.com/Program-AR/pilas-bloques-react/actions/runs/4467530026/jobs/7847128938#step:7:19
Está comentada la linea todavía:
https://github.com/Program-AR/pilas-bloques-react/blob/8b999dffa22c156ffed863b2e5eb512b3e437b58/scripts/package.sh#L100
(se me chanfleó descomentarla)
¿podés probar de nuevo @dlopezalvas porfa?
src/components/home/BookCards.tsx
Outdated
| @@ -1,4 +1,4 @@ | |||
| import { Card, CardActionArea, Stack, Typography } from "@mui/material" | |||
| import { Card, Stack, Typography } from "@mui/material" | |||
There was a problem hiding this comment.
Arrancar siendo estrictos está perfecto. Siempre se puede decidir dejar de serlo.
| render(<App />); | ||
| const linkElement = screen.getByText(/learn react/i); | ||
| expect(linkElement).toBeInTheDocument(); | ||
| expect(true).toBeTruthy(); |
There was a problem hiding this comment.
Acá me hubiera gustado tener un test de render pavo más que un test de truthiness pavo, pero ya vendrá. 😉
|
De paso hay q resolver conflicts :P |
|
Ya empaqueta el .dmg:) |
Resolves Program-AR/pilas-bloques-ember#1256