Skip to content
This repository has been archived by the owner on Nov 29, 2023. It is now read-only.

Prepare 3.0.1 #186

Merged
merged 1 commit into from
Aug 18, 2023
Merged

Prepare 3.0.1 #186

merged 1 commit into from
Aug 18, 2023

Conversation

eyelidlessness
Copy link
Contributor

@eyelidlessness eyelidlessness commented Aug 18, 2023

  • No vulnerabilities reported by npm audit in project or demo.
  • Pretty sure I addressed everything reported by Dependabot. It would be nice if there was a way to validate this before merging?
  • Other dependencies updated
    • Fixed: removed @ts-expect-error in src/dom/node/Node.ts, as TypeScript now supports different getter/setter types.
    • Added note about threads/singleThread config API change for future Vitest updates

Note: Vitest was not updated past 0.28.5, because newer versions would update the snapshot format.

- No vulnerabilities reported by `npm audit` in project or demo.
- Pretty sure I addressed everything reported by Dependabot. It would be nice if there was a way to validate this before merging?
- Other dependencies updated
  - Fixed: removed `@ts-expect-error` in src/dom/node/Node.ts, as TypeScript now supports different getter/setter types.
  - Added note about `threads`/`singleThread` config API change for future Vitest updates
@eyelidlessness eyelidlessness marked this pull request as ready for review August 18, 2023 22:16
// also caused problems with changes to the
// snapshot format.
// singleThread: true,

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This isn't really related. But it took a fair bit of time to figure out, and I'd hate to repeat that if/when we update Vitest again in the future.

@lognaturel lognaturel merged commit 5d12ef4 into master Aug 18, 2023
10 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants