Skip to content

bytes-put on aluasm!#74

Merged
dr-orlovsky merged 3 commits intoAluVM:masterfrom
6293:bytes-put
Jun 11, 2023
Merged

bytes-put on aluasm!#74
dr-orlovsky merged 3 commits intoAluVM:masterfrom
6293:bytes-put

Conversation

@6293
Copy link
Contributor

@6293 6293 commented Jun 9, 2023

No description provided.

@codecov-commenter
Copy link

codecov-commenter commented Jun 9, 2023

Codecov Report

Merging #74 (3df142a) into master (a49997e) will decrease coverage by 0.5%.
The diff coverage is 100.0%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@           Coverage Diff            @@
##           master     #74     +/-   ##
========================================
- Coverage    41.5%   41.1%   -0.5%     
========================================
  Files          19      19             
  Lines        4635    4542     -93     
========================================
- Hits         1925    1865     -60     
+ Misses       2710    2677     -33     
Impacted Files Coverage Δ
src/isa/instr.rs 10.5% <ø> (+2.3%) ⬆️
src/lib.rs 15.5% <ø> (ø)
src/isa/exec.rs 59.9% <100.0%> (-3.9%) ⬇️

... and 4 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

Copy link
Member

@dr-orlovsky dr-orlovsky left a comment

Choose a reason for hiding this comment

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

It seems like we have a failure in CI related to the change in feature gates in this PR

@6293
Copy link
Contributor Author

6293 commented Jun 11, 2023

fixed

Copy link
Member

@dr-orlovsky dr-orlovsky left a comment

Choose a reason for hiding this comment

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

ACK 3df142a

@dr-orlovsky dr-orlovsky merged commit 4514db9 into AluVM:master Jun 11, 2023
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