Skip to content

abi compatible data types feature request #1375

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
merged 1 commit into from
Sep 1, 2025
Merged

abi compatible data types feature request #1375

merged 1 commit into from
Sep 1, 2025

Conversation

qor-lb
Copy link
Contributor

@qor-lb qor-lb commented Jul 6, 2025

closes #917

@github-actions GitHub Actions
Copy link

github-actions bot commented Jul 6, 2025

The created documentation from the pull request is available at: docu-html

alex-warg-mb
alex-warg-mb reviewed Jul 9, 2025
View reviewed changes
4og
4og reviewed Jul 18, 2025
View reviewed changes
@HartmannNico HartmannNico mentioned this pull request Jul 21, 2025
Open
LittleHuba
LittleHuba reviewed Jul 21, 2025
View reviewed changes
4og
4og reviewed Jul 28, 2025
View reviewed changes
@LakmalBMW
Copy link

LakmalBMW commented Aug 6, 2025 •
edited
Loading

@awillenbuecher-xq-tec Could you see the current status of the open review comments and see what else is pending to get the first version of this ready to be reviewed by the architecture round?

LittleHuba
LittleHuba reviewed Aug 13, 2025
View reviewed changes
@awillenbuecher-xq-tec
Copy link
Contributor

@awillenbuecher-xq-tec Could you see the current status of the open review comments and see what else is pending to get the first version of this ready to be reviewed by the architecture round?

@LakmalBMW All comments have been answered and all requested changes implemented, I think.

LittleHuba
LittleHuba previously approved these changes Aug 14, 2025
View reviewed changes
@johannes-esr
Copy link
Contributor

@qor-lb, @awillenbuecher-xq-tec, can you please resolve the merge conflicts?

@qor-lb qor-lb force-pushed the abi_types_feature_request branch 3 times, most recently from 3f07f99 to 91180b9 Compare August 26, 2025 09:07
FScholPer
FScholPer approved these changes Sep 1, 2025
View reviewed changes
Copy link
Contributor

@FScholPer FScholPer left a comment

Choose a reason for hiding this comment

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

Looks okay for me

@qor-lb qor-lb merged commit ed8a79d into main Sep 1, 2025
8 checks passed
@qor-lb qor-lb deleted the abi_types_feature_request branch September 1, 2025 11:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@HartmannNico HartmannNico HartmannNico left review comments

@FScholPer FScholPer FScholPer approved these changes

@LittleHuba LittleHuba LittleHuba left review comments

@arsibo arsibo Awaiting requested review from arsibo

@johannes-esr johannes-esr Awaiting requested review from johannes-esr johannes-esr is a code owner

@antonkri antonkri Awaiting requested review from antonkri antonkri is a code owner

@markert-r markert-r Awaiting requested review from markert-r

+3 more reviewers

@4og 4og 4og left review comments

@awillenbuecher-xq-tec awillenbuecher-xq-tec awillenbuecher-xq-tec approved these changes

@alex-warg-mb alex-warg-mb alex-warg-mb left review comments

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
feature_request
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Feature Request for ABI compatible datatypes
9 participants
@qor-lb @LakmalBMW @awillenbuecher-xq-tec @johannes-esr @4og @LittleHuba @HartmannNico @FScholPer @alex-warg-mb