Skip to content

Fix Sorbet types#28

Merged
redox merged 1 commit intomainfrom
su/fix-sorbet-types
Mar 9, 2026
Merged

Fix Sorbet types#28
redox merged 1 commit intomainfrom
su/fix-sorbet-types

Conversation

@redox
Copy link
Member

@redox redox commented Mar 9, 2026

Tricky to spot until you actually depends on them from another project with tapioca...

Tricky to spot until you actually depends on them from another project
with `tapioca`...
@redox redox requested a review from francoischalifour March 9, 2026 19:45
Copy link
Contributor

@albert20260301 albert20260301 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 fix! Using the fully qualified namespace makes total sense to avoid conflicts/ambiguities. Approved!

Copy link
Contributor

@albert20260301 albert20260301 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 fix! Using the fully qualified namespace Altertable::Adapters::Response makes total sense to avoid conflicts/ambiguities. Approved!

@redox redox merged commit 75f6b7f into main Mar 9, 2026
6 checks passed
@redox redox deleted the su/fix-sorbet-types branch March 9, 2026 20:29
@redox redox mentioned this pull request Mar 9, 2026
redox added a commit that referenced this pull request Mar 9, 2026
#28 didn't follow
conventional commits so nothing got released
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.

3 participants