Skip to content

Bump pFUnit version#238

Merged
danielhollas merged 3 commits intomasterfrom
bump-pfunit
May 1, 2026
Merged

Bump pFUnit version#238
danielhollas merged 3 commits intomasterfrom
bump-pfunit

Conversation

@danielhollas
Copy link
Copy Markdown
Contributor

We've been pinning the pFUnit version to an old commit for GCC 7 support. But we no longer test GCC 7 so let's update to the latest official version. (this is also a pre-requisite for compilation on MacOS, see #219 )

Also fixed shellcheck linter violations in all scripts in dev_scripts directory.

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 30, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.06%. Comparing base (fbbdb89) to head (35f7099).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #238   +/-   ##
=======================================
  Coverage   84.06%   84.06%           
=======================================
  Files          47       47           
  Lines        6613     6613           
  Branches      766      766           
=======================================
  Hits         5559     5559           
  Misses        855      855           
  Partials      199      199           
Flag Coverage Δ
unittests 18.37% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@danielhollas danielhollas requested a review from JanosJiri April 30, 2026 17:14
@danielhollas danielhollas merged commit b824053 into master May 1, 2026
16 checks passed
@danielhollas danielhollas deleted the bump-pfunit branch May 1, 2026 09:05
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