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

ci: add test on big-endian arch in CI #26

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

ci: add test on big-endian arch in CI #26

wants to merge 2 commits into from

Conversation

vthib
Copy link
Owner

@vthib vthib commented Feb 26, 2023

No description provided.

@vthib vthib force-pushed the add-ci-s390x branch 4 times, most recently from 67d5d6c to 941a70b Compare February 26, 2023 17:00
@vthib
Copy link
Owner Author

vthib commented Feb 26, 2023

Tests are failing due to endianness issues in YARA, see VirusTotal/yara#1884

@vthib vthib force-pushed the add-ci-s390x branch 7 times, most recently from 3b2a7b7 to 18e9d2e Compare August 27, 2023 17:36
@vthib vthib mentioned this pull request Sep 10, 2023
@vthib vthib force-pushed the add-ci-s390x branch 2 times, most recently from ec05d1f to 637f57e Compare April 15, 2024 22:50
Copy link

codecov bot commented Oct 20, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.59%. Comparing base (9d70497) to head (a63c4cb).
Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master      #26   +/-   ##
=======================================
  Coverage   98.59%   98.59%           
=======================================
  Files          81       81           
  Lines       24558    24558           
=======================================
  Hits        24214    24214           
  Misses        344      344           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

vthib added 2 commits October 20, 2024 23:31
Do not do the check inside the C callback, properly save the value
then compare it.
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.

1 participant