Skip to content

fix(Python): missing Component active binding#1251

Merged
BotellaA merged 1 commit intonextfrom
fix/python-active
Mar 20, 2026
Merged

fix(Python): missing Component active binding#1251
BotellaA merged 1 commit intonextfrom
fix/python-active

Conversation

@BotellaA
Copy link
Member

No description provided.

@github-actions
Copy link
Contributor

Cpp-Linter Report ⚠️

Some files did not pass the configured checks!

clang-tidy (v20.1.8) reports: 3 concern(s)
  • bindings/python/src/model/mixin/core/component.cpp:39:10: warning: [misc-use-internal-linkage]

    function 'define_component' can be made static or moved into an anonymous namespace to enforce internal linkage

       39 |     void define_component( pybind11::module& module )
          |          ^
          |     static 
  • bindings/python/src/model/mixin/core/component.cpp:41:9: warning: [cppcoreguidelines-pro-bounds-pointer-arithmetic]

    do not use pointer arithmetic

       41 |         PYTHON_COMPONENT( 2 );
          |         ^
    /__w/OpenGeode/OpenGeode/bindings/python/src/model/mixin/core/component.cpp:30:21: note: expanded from macro 'PYTHON_COMPONENT'
       30 |         "Component" + std::to_string( dimension ) + "D";                       \
          |                     ^
  • bindings/python/src/model/mixin/core/component.cpp:42:9: warning: [cppcoreguidelines-pro-bounds-pointer-arithmetic]

    do not use pointer arithmetic

       42 |         PYTHON_COMPONENT( 3 );
          |         ^
    /__w/OpenGeode/OpenGeode/bindings/python/src/model/mixin/core/component.cpp:30:21: note: expanded from macro 'PYTHON_COMPONENT'
       30 |         "Component" + std::to_string( dimension ) + "D";                       \
          |                     ^

Have any feedback or feature suggestions? Share it here.

@BotellaA BotellaA merged commit 7a58198 into next Mar 20, 2026
24 checks passed
@BotellaA BotellaA deleted the fix/python-active branch March 20, 2026 08:28
@BotellaA
Copy link
Member Author

🎉 This PR is included in version 16.3.1-rc.2 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@BotellaA
Copy link
Member Author

🎉 This PR is included in version 16.3.1 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants