Skip to content

Modernize: replace boost::shared_ptr with std::shared_ptr in template… - #297

Draft
mirams wants to merge 3 commits into
masterfrom
modernize/std-shared-ptr
Draft

mirams wants to merge 3 commits into
masterfrom
modernize/std-shared-ptr

Conversation

@mirams

@mirams mirams commented Apr 30, 2026 •

Copy link
Copy Markdown
Collaborator

…s and reference models

Also add #include to the shared hpp includes template and all reference .hpp files so generated code compiles without boost/shared_ptr.hpp.

Description

Motivation and Context

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Documentation checklist

  • I have updated all documentation in the code where necessary.
  • I have checked spelling in all (new) comments and documentation.
  • I have added a note to RELEASE.md if relevant (new feature, breaking change, or notable bug fix).
  • I have updated version & citation.txt & citation.cff version.

Testing

  • Testing is done automatically and codecov shows test coverage
  • This cannot be tested automatically

mirams and others added 2 commits April 30, 2026 16:06
…s and reference models

Also add #include <memory> to the shared hpp includes template and all
reference .hpp files so generated code compiles without boost/shared_ptr.hpp.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@codecov

codecov Bot commented Sep 2, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (7878bf3) to head (9f1670c).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff            @@
##            master      #297   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           30        30           
  Lines         1789      1789           
=========================================
  Hits          1789      1789           

☔ View full report in Codecov by Harness.
📢 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.

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