Skip to content

Conversation

martakam
Copy link
Member

@martakam martakam commented Nov 8, 2026 •

This PR utilizes CI properly (finally).

We have checks for (both frontend and backend):

  • Formatting with Prettier
  • Linting with ESLint
  • Building the project correctly

For the frontend, we also run all tests.

This PR is to be merged after #111, since it builds upon that one.

Closes #110

@martakam martakam self-assigned this Nov 8, 2026
@martakam martakam added the enhancement New feature or request label Nov 8, 2026
@martakam martakam added this to the Deliverable 4 milestone Nov 8, 2026
@martakam
Copy link
Member Author

martakam commented Nov 8, 2026

Finally, green checkmarks! 🎉

I also added caching in cdc546b, which is confirmed by the logs:

Packages: +528
+++
Progress: resolved 528, reused 528, downloaded 0, added 528, done

This PR is now ready for review, although it will be easier to review after #111 is merged into main, because the diff will be easier to read.

@martakam martakam marked this pull request as ready for review November 8, 2026 18:31
@martakam martakam requested a review from a team as a code owner November 8, 2026 18:31
@johahor
Copy link
Member

johahor commented Nov 10, 2026

Nice work @martakam. Thanks for going above and beyond and setting this up.

After my merge en commit i got to see the CI in action as well. Good call on keeping E2E tests out of the CI.. took quite some time just the ones we now have 😅

@johahor johahor merged commit 9345222 into main Nov 10, 2026
7 checks passed
@martakam martakam deleted the ci/formatting-linting branch November 12, 2026 21:31
Sign in to join this conversation on GitHub.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ci: add CI for formatting and linting
2 participants