Skip to content

Migrate signature help to Rubydex - #4098

Merged
vinistock merged 1 commit into
rubydex_adoption_feature_branchfrom
vs_migrate_signature_help_to_rubydex
May 8, 2026
Merged

Migrate signature help to Rubydex#4098
vinistock merged 1 commit into
rubydex_adoption_feature_branchfrom
vs_migrate_signature_help_to_rubydex

Conversation

@vinistock

Copy link
Copy Markdown
Member

Motivation

Migrate signature help to Rubydex.

Implementation

Matched the same logic we had for the old indexer signatures, just using the Rubydex graph.

Automated Tests

Tests are the same except for the RBS overload ones. Instead of trying to index the real RBS core definitions, we can just index a single hand written source which is much faster.

@vinistockvinistock self-assigned this May 8, 2026
@vinistock
vinistock requested a review from a team as a code ownerMay 8, 2026 14:01
@vinistockvinistock added server This pull request should be included in the server gem's release notes other Changes that aren't bugfixes, enhancements or breaking changes labels May 8, 2026
@vinistock
vinistockforce-pushed the vs_migrate_completion_to_rubydex branch from 07c1526 to e6c4e7aCompareMay 8, 2026 14:03
@vinistock
vinistockforce-pushed the vs_migrate_signature_help_to_rubydex branch from 4cbd6d9 to 0edb1e8CompareMay 8, 2026 14:03
@vinistockvinistock mentioned this pull request May 8, 2026
19 tasks
@vinistock
vinistockforce-pushed the vs_migrate_completion_to_rubydex branch from e6c4e7a to 447395cCompareMay 8, 2026 16:10
Base automatically changed from vs_migrate_completion_to_rubydex to rubydex_adoption_feature_branchMay 8, 2026 17:17
@vinistock
vinistockforce-pushed the vs_migrate_signature_help_to_rubydex branch from 0edb1e8 to b3e7d40CompareMay 8, 2026 17:18
@vinistock
vinistock merged commit 6fa9320 into rubydex_adoption_feature_branchMay 8, 2026
34 checks passed
@vinistock
vinistock deleted the vs_migrate_signature_help_to_rubydex branch May 8, 2026 17:18
vinistock added a commit that referenced this pull request Jun 11, 2026
vinistock added a commit that referenced this pull request Jun 15, 2026
vinistock added a commit that referenced this pull request Jul 2, 2026
vinistock added a commit that referenced this pull request Jul 3, 2026
vinistock added a commit that referenced this pull request Jul 24, 2026
vinistock added a commit that referenced this pull request Aug 21, 2026
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

otherChanges that aren't bugfixes, enhancements or breaking changesserverThis pull request should be included in the server gem's release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@vinistock@Morriar