Skip to content

Index globals with RBS indexer#616

Merged
alexcrocha merged 1 commit intomainfrom
02-27-index_globals_with_rbs_indexer
Mar 3, 2026
Merged

Index globals with RBS indexer#616
alexcrocha merged 1 commit intomainfrom
02-27-index_globals_with_rbs_indexer

Conversation

@alexcrocha
Copy link
Contributor

@alexcrocha alexcrocha commented Feb 27, 2026

One more step towards #87.

This PR adds visit_global_node so global variable declarations ($foo: String) are discovered and resolved as members of Object, matching Ruby indexer's behaviour.

Copy link
Contributor Author

alexcrocha commented Feb 27, 2026

@alexcrocha alexcrocha force-pushed the 02-27-index_globals_with_rbs_indexer branch from cc6bbac to f77fbfe Compare February 27, 2026 23:49
@alexcrocha alexcrocha self-assigned this Feb 27, 2026
@alexcrocha alexcrocha added the enhancement New feature or request label Feb 27, 2026
@alexcrocha alexcrocha marked this pull request as ready for review February 27, 2026 23:50
@alexcrocha alexcrocha requested a review from a team as a code owner February 27, 2026 23:50
Copy link
Contributor

@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.

Looks good except the multi-line comment handling, as commented in #615.

  This adds visit_global_node so global variable declarations
  (e.g., $foo: String) are discovered and resolved as members
  of Object
@alexcrocha alexcrocha force-pushed the 02-27-index_globals_with_rbs_indexer branch from f77fbfe to f7e2542 Compare March 3, 2026 03:45
@alexcrocha alexcrocha force-pushed the 02-27-index_constants_with_rbs_indexer branch from ed6eba1 to 60b8df1 Compare March 3, 2026 03:45
Base automatically changed from 02-27-index_constants_with_rbs_indexer to main March 3, 2026 03:54
@alexcrocha alexcrocha merged commit d1cc7cc into main Mar 3, 2026
50 checks passed
@alexcrocha alexcrocha deleted the 02-27-index_globals_with_rbs_indexer branch March 3, 2026 03:55
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.

3 participants