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(runtime): move InstanceConfig to lib/runtime/wasmer #2751

Merged
merged 2 commits into from
Aug 16, 2022

Conversation

qdm12
Copy link
Contributor

@qdm12 qdm12 commented Aug 15, 2022

Changes

Move InstanceConfig from lib/runtime to Config in lib/runtime/wasmer

This is so private fields can be added for test-only in the wasmer package (blocks #2747)

Tests

Issues

#2418

Primary Reviewer

@timwu20

@qdm12 qdm12 marked this pull request as ready for review August 15, 2022 14:27
@qdm12 qdm12 force-pushed the qdm12/runtime/move-config-wasmer branch from 997e201 to c28ae53 Compare August 15, 2022 14:28
@qdm12 qdm12 added the PR Easy label Aug 15, 2022
@codecov
Copy link

codecov bot commented Aug 15, 2022

Codecov Report

Merging #2751 (f0120a7) into development (2444d0b) will increase coverage by 0.15%.
The diff coverage is 90.90%.

Additional details and impacted files
@@               Coverage Diff               @@
##           development    #2751      +/-   ##
===============================================
+ Coverage        62.78%   62.94%   +0.15%     
===============================================
  Files              213      213              
  Lines            27081    27081              
===============================================
+ Hits             17003    17045      +42     
+ Misses            8527     8486      -41     
+ Partials          1551     1550       -1     

@qdm12 qdm12 force-pushed the qdm12/runtime/move-config-wasmer branch from c28ae53 to 796fb65 Compare August 15, 2022 19:35
@qdm12 qdm12 force-pushed the qdm12/runtime/move-config-wasmer branch 2 times, most recently from 28edafa to 6e57e0f Compare August 15, 2022 21:13
@qdm12 qdm12 force-pushed the qdm12/runtime/move-config-wasmer branch from 6e57e0f to f0120a7 Compare August 15, 2022 23:33
@qdm12 qdm12 merged commit 8b0f6f0 into development Aug 16, 2022
@qdm12 qdm12 deleted the qdm12/runtime/move-config-wasmer branch August 16, 2022 00:04
@github-actions
Copy link

🎉 This PR is included in version 0.7.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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.

3 participants