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

chore: github actions workflow for tests / visual regressions #21

Merged
merged 10 commits into from
Dec 16, 2024
Merged

Conversation

holloway
Copy link
Contributor

@holloway holloway commented Dec 12, 2024

  • Add CI workflow for tests (vitest, TS, lint)
  • Moves generated JSON into the generated directory
  • Renames vis-reg-test.yml to the more readable visual-regression-test.yml
  • Temporarily commits the api client to Git (see comment below)

@holloway
Copy link
Contributor Author

holloway commented Dec 12, 2024

@jennifer-richards now that we've got unit tests I've added a workflow to run those, TS, and linting on CI.

Because the Nuxt build and TS require the generated API client we'll need to have the client on CI. It would be best to generate that on CI so that we have current types, but as we can't do that yet I've temporarily commited the API client (which is a single file) to test this workflow.

@holloway holloway marked this pull request as ready for review December 12, 2024 21:52
- name: Setup Node
uses: actions/setup-node@v3
with:
node-version: 23.x
Copy link
Member

Choose a reason for hiding this comment

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

I would avoid using non-LTS versions in anything unless absolutely necessary.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah it was the quickest fix for this... I'll look for a workaround
https://github.com/ietf-tools/rfced-www/actions/runs/12311686063/job/34362303820

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Done

Copy link
Member

@jennifer-richards jennifer-richards left a comment

Choose a reason for hiding this comment

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

lgtm

@holloway holloway merged commit 401e5bc into main Dec 16, 2024
3 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.

3 participants