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: update @ts-ast-parser/core to support node 21 #180

Merged
merged 4 commits into from
Mar 18, 2024

Conversation

avivash
Copy link
Member

@avivash avivash commented Mar 18, 2024

Description

Installing every-cli on node 21 was breaking due to the @ts-ast-parser/core version in @fission-codes/homestar. Bumping to 0.8.0, which should work with node 21

updating wit.js to use the new TypeLiteral type for plain objects since ObjectLiteral has been deprecated

removed @ipld/dag-cbor from ucan package because it was unused and causing depcheck to fail

Type of change

  • Bug fix (non-breaking change that fixes an issue)

@avivash avivash self-assigned this Mar 18, 2024
@avivash avivash requested a review from hugomrdias as a code owner March 18, 2024 20:30
@avivash avivash requested a review from matheus23 March 18, 2024 20:30
Copy link

socket-security bot commented Mar 18, 2024

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/@ts-ast-parser/[email protected] Transitive: filesystem +22 1.37 MB jmarimon

🚮 Removed packages: npm/@ts-ast-parser/[email protected]

View full report↗︎

Copy link
Member

@walkah walkah left a comment

Choose a reason for hiding this comment

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

:shipit:

@avivash avivash merged commit 65e8e6c into main Mar 18, 2024
8 checks passed
@avivash avivash deleted the avivash/update-ts-ast-parser-core-to-support-node-21 branch March 18, 2024 21:07
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