Skip to content

Conversation

@marcosbento
Copy link
Contributor

Description

Modernise and consolidate the code base by systematically using using instead of typedef to introduce type aliases.

Contributor Declaration

By opening this pull request, I affirm the following:

  • All authors agree to the Contributor License Agreement.
  • The code follows the project's coding standards.
  • I have performed self-review and added comments where needed.
  • I have added or updated tests to verify that my changes are effective and functional.
  • I have run all existing tests and confirmed they pass.

@codecov-commenter
Copy link

codecov-commenter commented Dec 2, 2025

Codecov Report

❌ Patch coverage is 64.28571% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 65.81%. Comparing base (f9167ba) to head (c97993d).

Files with missing lines Patch % Lines
src/eckit/os/SharedInt.cc 0.00% 2 Missing ⚠️
src/eckit/sql/SQLTable.cc 0.00% 2 Missing ⚠️
src/eckit/io/Offset.cc 0.00% 1 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #239   +/-   ##
========================================
  Coverage    65.81%   65.81%           
========================================
  Files         1133     1133           
  Lines        57484    57484           
  Branches      4340     4340           
========================================
  Hits         37833    37833           
  Misses       19651    19651           

☔ 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.

@marcosbento marcosbento force-pushed the feature/replace_typedef branch from cff21c5 to bc24ea4 Compare December 2, 2025 18:43
@marcosbento marcosbento force-pushed the feature/replace_typedef branch from bc24ea4 to 9251415 Compare December 4, 2025 10:29
@marcosbento marcosbento marked this pull request as ready for review December 4, 2025 11:25
@Ozaq Ozaq force-pushed the feature/replace_typedef branch from 9251415 to c97993d Compare December 4, 2025 14:42
@Ozaq Ozaq merged commit 8f65437 into develop Dec 4, 2025
255 of 260 checks passed
@Ozaq Ozaq deleted the feature/replace_typedef branch December 4, 2025 16:20
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.

5 participants