Skip to content

Conversation

@mergify
Copy link
Contributor

@mergify mergify bot commented May 8, 2025

  1. Rename args to msg consistently in WasmEngine interface
  2. Rename file to wasm_engine.go to avoid the term "wasmer" in the wasmd codebase
    This is an automatic backport of pull request Rename message args 💅 #2207 done by Mergify.

* Rename args to `msg` consistently in WasmEngine interface

* Rename file to wasm_engine.go

"Wasmer" is an implementation detail that can change at any time

(cherry picked from commit 1756d2d)
@mergify mergify bot mentioned this pull request May 8, 2025
@chipshort chipshort merged commit 5e1248a into releases/v0.6x May 8, 2025
12 of 14 checks passed
@chipshort chipshort deleted the mergify/bp/releases/v0.6x/pr-2207 branch May 8, 2025 14:45
@codecov
Copy link

codecov bot commented May 8, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 45.96%. Comparing base (6655b4a) to head (03f4952).
Report is 14 commits behind head on releases/v0.6x.

Additional details and impacted files

Impacted file tree graph

@@                Coverage Diff                 @@
##           releases/v0.6x    #2247      +/-   ##
==================================================
- Coverage           45.98%   45.96%   -0.02%     
==================================================
  Files                  78       78              
  Lines               10684    10684              
==================================================
- Hits                 4913     4911       -2     
- Misses               5334     5335       +1     
- Partials              437      438       +1     
Files with missing lines Coverage Δ
x/wasm/types/wasm_engine.go 0.00% <ø> (ø)

... and 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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