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

Fix upload action and missing protoc #821

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

tmpolaczyk
Copy link
Contributor

@tmpolaczyk tmpolaczyk commented Jan 24, 2025

Update upload-artifact action to v4 and add missing protobuf-compiler dependency

Copy link
Contributor

WASM runtime size check:

Compared to target branch

dancebox runtime: 1420 KB (no changes) ✅

flashbox runtime: 824 KB (no changes) ✅

dancelight runtime: 2168 KB (+2168 KB) ⚠️

container chain template simple runtime: 1124 KB (-4296 KB) ✅

container chain template frontier runtime: 1400 KB (-5152 KB) ✅

Copy link
Contributor

Coverage Report

(master)

@@                   Coverage Diff                   @@
##           master   tomasz-upload-protoc     +/-   ##
=======================================================
  Coverage   65.76%                 65.76%   0.00%     
  Files         332                    332             
  Lines       58501                  58501             
=======================================================
  Hits        38472                  38472             
  Misses      20029                  20029             
Files Changed Coverage

Coverage generated Fri Jan 24 11:27:13 UTC 2025

@girazoki
Copy link
Contributor

please write a description

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