Skip to content

Add addObject spelling support (again)#322

Merged
alxbilger merged 2 commits intosofa-framework:masterfrom
CRIStAL-PADR:pr-add-difflib-support-fix
Dec 8, 2022
Merged

Add addObject spelling support (again)#322
alxbilger merged 2 commits intosofa-framework:masterfrom
CRIStAL-PADR:pr-add-difflib-support-fix

Conversation

@damienmarchal
Copy link
Copy Markdown
Contributor

@damienmarchal damienmarchal commented Dec 7, 2022

Same as #314 but with 2.6.0 setting for the backward compatibility to old pybind11 version.
py::type was introduced at 2.6.0 so there is no need for the macro after that.

The previous merged problem was not detected before because pybind11 on sofa-jenkins is 2.6.2 while pybind11 on sofapython/github actions is 2.4.

@damienmarchal damienmarchal force-pushed the pr-add-difflib-support-fix branch from 20a1956 to fc46102 Compare December 7, 2022 20:10
@damienmarchal damienmarchal force-pushed the pr-add-difflib-support-fix branch from fc46102 to be841c8 Compare December 7, 2022 20:13
@damienmarchal damienmarchal changed the title FIX Difflib support Add addObject spelling support (again) Dec 7, 2022
Copy link
Copy Markdown
Contributor

@alxbilger alxbilger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Almost good to go

Comment thread Plugin/src/SofaPython3/SpellingSuggestionHelper.h Outdated
Thanks

Co-authored-by: Alex Bilger <alxbilger@users.noreply.github.com>
@damienmarchal
Copy link
Copy Markdown
Contributor Author

Done

Copy link
Copy Markdown
Contributor

@alxbilger alxbilger left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I mark this PR ready

@alxbilger alxbilger merged commit 7a44ca6 into sofa-framework:master Dec 8, 2022
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.

2 participants