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: update test file for jenkins #420

Merged
merged 1 commit into from
Feb 29, 2024
Merged

ci: update test file for jenkins #420

merged 1 commit into from
Feb 29, 2024

Conversation

Smeds
Copy link
Collaborator

@Smeds Smeds commented Feb 29, 2024

the result for one variant and the f1f2 field differs when you run the analysis on marvin and jenkins. Unknown why, but this new checksum will make sure that the test passes on jenkins.

This PR:

(If this is a release PR, no need to add following. Leave this part empty)
(Use the following lines to create a PR text body. Make sure to remove all non-relevant one after you're done)
(Repeat each field as many times as necessary)

Added: for new features.
Changed: for changes in existing functionality.
Deprecated: for soon-to-be removed features.
Removed: for now removed features.
Fixed: for any bug fixes.
Security: in case of vulnerabilities.

Review and tests:

  • Tests pass
  • If you've fixed a bug or added code that should be tested, add tests!
  • Code review
  • CHANGELOG.md is updated
  • New code is executed and covered by tests, and test approve

the result for one variant and the f1f2 field differs when you run the analysis on marvin and jenkins. Unknown why, but this new checksum will make sure that the test passes on jenkins.
@Smeds Smeds requested a review from jonca79 as a code owner February 29, 2024 12:10
@Smeds Smeds merged commit 03203b9 into develop Feb 29, 2024
9 of 10 checks passed
@Smeds Smeds deleted the Smeds-patch-1 branch February 29, 2024 12:15
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