Skip to content

Conversation

@domingo2000
Copy link
Contributor

@domingo2000 domingo2000 commented Jun 1, 2025

Closes #377

Add Go To Definition for Action::Mailbox callbacks.

Screencast.from.01-06-25.12.28.39.webm

@domingo2000 domingo2000 requested a review from a team as a code owner June 1, 2025 16:27
@domingo2000 domingo2000 force-pushed the add-definition-on-mailbox-callbacks branch from cc32523 to b5f4cc8 Compare June 2, 2025 17:26
@vinistock vinistock added the enhancement New feature or request label Jul 15, 2025
Copy link
Member

@vinistock vinistock left a comment

Choose a reason for hiding this comment

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

Thanks for the contribution! This looks great.

For some weird reason, CI seems to be stuck. Can you please rebase the PR?

@domingo2000 domingo2000 force-pushed the add-definition-on-mailbox-callbacks branch from b5f4cc8 to 3a04173 Compare July 17, 2025 02:01
@domingo2000
Copy link
Contributor Author

domingo2000 commented Jul 17, 2025

Just rebased, windows build still failing 😞.

@vinistock
Copy link
Member

Yes, we're getting hit by sparklemotion/sqlite3-ruby#628. I'm not exactly sure what is the solution yet.

@vinistock
Copy link
Member

Thank you for the contribution!

@vinistock vinistock merged commit 7fe21f6 into Shopify:main Jul 17, 2025
12 of 16 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support ActionMailbox callbacks for Definition requests

2 participants