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

Development Strict Mode #1006

Merged
merged 10 commits into from
Dec 30, 2024
Merged

Development Strict Mode #1006

merged 10 commits into from
Dec 30, 2024

Conversation

refringe
Copy link
Contributor

  • Enables TS strict mode
  • Adds script to check types
  • Adds workflow to run check-type script
  • Updates the code-checking workflows to share the same set-up job

This updates the tsconfig.json option to enable strict mode. However, we use TSX for development and SWC for release, which do not type-check, so this option only gives additional linting/visual-feedback in VSCode.

Additionally, I've added a NPM script npm run lint:types and a GitHub workflow that runs it.

This depends on #1005.

- Updates version numbers in `.nvmrc`, `package.json`, and `Server.code-workspace`
- Updates `@yao-pkg/pkg`, `@yao-pkg/pkg-fetch`, and `@types/node` to latest
- Updates JSON import to FS read operation in `gulpfile.mjs`
- Updates node build target in `gulpfile.mjs`
- Enables TS strict mode
- Adds script to check types
- Adds workflow to run check-type script
- Updates the code-checking workflows to share the same set-up job

This updates the `tsconfig.json` option to enable strict mode. *However*, we use TSX for development and SWC for release, which do not type-check, so this option only gives additional linting/visual-feedback in VSCode.

Additionally, I've added a NPM script `npm run lint:types` and a GitHub workflow that runs it.
@refringe refringe added the enhancement New feature or request label Dec 30, 2024
@refringe refringe added this to the 4.0.0 milestone Dec 30, 2024
@refringe refringe requested a review from chompDev December 30, 2024 04:54
@refringe refringe self-assigned this Dec 30, 2024
Updates the upload-artifact action to v4. Specifies some additional options.
Updates the download-artifact action to v4.
Updates the NPM scripts to use NPX to call vitest scripts to help issues in CI.
Reverts to original workflow design and implements NPM package caching.
@refringe
Copy link
Contributor Author

I'm done messing with the workflows. Definitely squash this one. 😄

@chompDev chompDev merged commit 4afc2f5 into 4.0.0-DEV Dec 30, 2024
0 of 6 checks passed
@chompDev chompDev deleted the strict-mode branch December 30, 2024 09:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants