Skip to content

Remove libprimac const definition#40

Merged
amontoison merged 1 commit into
libprima:mainfrom
andreasnoack:an/relocate
Jul 14, 2025
Merged

Remove libprimac const definition#40
amontoison merged 1 commit into
libprima:mainfrom
andreasnoack:an/relocate

Conversation

@andreasnoack
Copy link
Copy Markdown
Contributor

This definition is not needed and it breaks relocatability of PRIMA when built into a system image.

This definition is not needed and it breaks relocatability of PRIMA
when built into a system image.
@amontoison amontoison merged commit b7099d1 into libprima:main Jul 14, 2025
12 of 16 checks passed
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Jul 14, 2025

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 86.71%. Comparing base (6bc54d7) to head (c7bb1fd).
Report is 20 commits behind head on main.

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@            Coverage Diff             @@
##             main      #40      +/-   ##
==========================================
- Coverage   87.50%   86.71%   -0.79%     
==========================================
  Files           4        4              
  Lines         320      301      -19     
==========================================
- Hits          280      261      -19     
  Misses         40       40              

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

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

@andreasnoack andreasnoack deleted the an/relocate branch July 14, 2025 17:49
@amontoison
Copy link
Copy Markdown
Member

@andreasnoack I did a new release 0.2.4 with this modification.

@andreasnoack
Copy link
Copy Markdown
Contributor Author

Great. Thanks for the fast processing.

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