Skip to content

update wasm related crates#32

Merged
BrytonLee merged 2 commits intoOptimatistOpenSource:mainfrom
serephus:update_incompatible_dependencies
Jun 3, 2024
Merged

update wasm related crates#32
BrytonLee merged 2 commits intoOptimatistOpenSource:mainfrom
serephus:update_incompatible_dependencies

Conversation

@serephus
Copy link
Contributor

@serephus serephus commented May 31, 2024

wasmtime and it's related crates have published 3 major releases in 3 months, there are some drastic changes in these releases, maybe we should update ASAP?

I've update some of usage of this changed APIs, but it require more changes to do things right. I plan to submit more PRs to adapt wasm related crates updates.

TODO:

  • fine-tuning error handling of wasm components (trappable_imports)
  • opentelemetry: using Gauge to replace ObservableGauge
  • async host?

@serephus serephus marked this pull request as ready for review May 31, 2024 09:27
Copy link
Member

@BrytonLee BrytonLee left a comment

Choose a reason for hiding this comment

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

I would like to see the upgradation of each dependency to be a separate PR. Please do that if you're also agree.

@serephus serephus marked this pull request as draft June 3, 2024 07:01
@serephus
Copy link
Contributor Author

serephus commented Jun 3, 2024

I would like to see the upgradation of each dependency to be a separate PR. Please do that if you're also agree.

done

@serephus serephus changed the title Update incompatible dependencies Update wasm related crates Jun 3, 2024
@serephus serephus changed the title Update wasm related crates update wasm related crates Jun 3, 2024
@serephus serephus marked this pull request as ready for review June 3, 2024 07:28
@serephus serephus requested a review from BrytonLee June 3, 2024 07:54
@BrytonLee BrytonLee merged commit c5bbe96 into OptimatistOpenSource:main Jun 3, 2024
@serephus serephus deleted the update_incompatible_dependencies branch June 3, 2024 08:48
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