Skip to content

Mark as ractor-safe #2041

New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom
Merged
merged 1 commit into from
Oct 3, 2024
Merged

Mark as ractor-safe #2041

merged 1 commit into from
Oct 3, 2024

Conversation

mohamedhafez
Copy link
Contributor

Closes #2024

@mohamedhafez mohamedhafez mentioned this pull request Oct 2, 2024
Open
soutaro
soutaro approved these changes Oct 3, 2024
View reviewed changes
Copy link
Member

@soutaro soutaro left a comment

Choose a reason for hiding this comment

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

💯

@soutaro soutaro added this to the RBS 3.6.x milestone Oct 3, 2024
@soutaro
Copy link
Member

soutaro commented Oct 3, 2024

Thank you @eregon for reviewing the source code to identify the extension is Ractor safe. (I think it's safe too.)

@soutaro soutaro added this pull request to the merge queue Oct 3, 2024
Merged via the queue into ruby:master with commit 9d0693b Oct 3, 2024
19 checks passed
@soutaro soutaro added the Released PRs already included in the released version label Oct 3, 2024
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@soutaro soutaro soutaro approved these changes

Assignees
No one assigned
Labels
Released PRs already included in the released version
Projects
None yet
Milestone
RBS 3.6.x
Development

Successfully merging this pull request may close these issues.

Make C-extension Ractor-safe
2 participants
@mohamedhafez @soutaro