Skip to content
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.

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

Make object option generic #2633

Merged
merged 14 commits into from
Sep 17, 2023
Merged

Make object option generic #2633

merged 14 commits into from
Sep 17, 2023

Conversation

tobbi
Copy link
Member

@tobbi tobbi commented Sep 9, 2023

"Sponges grow in the ocean. That just kills me. I wonder how much deeper the ocean would be if that didn't happen." - Steve Wright (from https://www.brainyquote.com/authors/steven-wright-quotes)

Fixes #2632

@MatusGuy
Copy link
Member

tobbi has the best commit messages

@tobbi tobbi marked this pull request as ready for review September 10, 2023 20:29
@tobbi tobbi requested review from MatusGuy, mrkubax10 and Vankata453 and removed request for MatusGuy September 11, 2023 19:01
Copy link
Member

@mrkubax10 mrkubax10 left a comment

Choose a reason for hiding this comment

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

Apply @Vankata453's suggestions and it will be good to merge.

@tobbi tobbi force-pushed the rework-object-options branch from 57fcc91 to 2132110 Compare September 13, 2023 20:48
@tobbi tobbi requested review from Rusty-Box and removed request for MatusGuy September 16, 2023 18:57
@tobbi tobbi merged commit b855913 into master Sep 17, 2023
@tobbi tobbi deleted the rework-object-options branch September 17, 2023 10:42
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.

Ability to get internal pointer from an object option
4 participants