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

[Cargo] Update deno dependencies (not the latest version) #93

Merged
merged 1 commit into from
Jul 17, 2024

Conversation

jaudiger
Copy link
Contributor

Part of #81

I didn't update the Deno dependencies to their latest version to reduce the possibility of regression due to breaking changes in the APIs. A lot of things have changed since version 0.201.0 of deno_core. The main modification in this PR is the usage of op2 in replacement of op (which was already deprecated in 0.201.0). op has been removed in favor of the new one. The new syntax is more verbose, but provides better constraining.

I updated it to 0.237.0 and kept in sync serde_v8 to 0.146.0. Starting version 0.238.0, there are other breaking changes that will be part of another PR.

I tested the modification with the following brioche commands:

brioche fmt -p packages/bat
brioche check -p packages/bat
brioche build -p packages/bat
brioche build -p packages/bat -e test
brioche install -p packages/bat

…e possibility of regression due to breaking changes in the APIs

Signed-off-by: jaudiger <[email protected]>
@jaudiger jaudiger changed the title build: update deno dependencies (not the latest version) [Cargo] Update deno dependencies (not the latest version) Jul 16, 2024
@kylewlacy kylewlacy merged commit a46843d into brioche-dev:main Jul 17, 2024
5 checks passed
@jaudiger jaudiger deleted the deno-dependencies-update branch July 17, 2024 07:37
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