Skip to content

Conversation

@cmenon12
Copy link
Member

@cmenon12 cmenon12 commented Jan 5, 2026

Description

  • Move the tsx dependency to the main dependencies, to avoid issues with Docker not finding tsx.
  • If there's a cache hit in the setup job of the code quality workflow, don't download it and don't run the rest of the steps.

Related issues: none

Checklist before marking as ready for review

Before marking as ready for review, please check that you have:

  • Added appropriate and passing tests for the changes I made.
  • Not broken any existing functionality or tests.
  • Not introduced any new linting or formatting errors.
  • Only added new dependencies that are necessary, do not introduce security vulnerabilities, and with tilde (~) version ranges.
  • Checked and resolved UI accessibility issues using Axe devtools. Unresolved issues are described above.
  • Updated the documentation if necessary.

Check the contributing guidelines for more details.

Screenshots (if appropriate)

This fixes an error with Docker: `tsx: command not found`
@cmenon12 cmenon12 self-assigned this Jan 5, 2026
@cmenon12 cmenon12 changed the title Move tsx to main dependencies; don't run superfluous workflow steps Move tsx to main dependencies; don't run superfluous code quality workflow setup steps Jan 5, 2026
@cmenon12 cmenon12 marked this pull request as ready for review January 5, 2026 10:06
@cmenon12 cmenon12 merged commit 5f348bb into main Jan 5, 2026
8 checks passed
@cmenon12 cmenon12 deleted the devops branch January 5, 2026 10:06
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