Skip to content

Restore a comment for Prism::Translation::Parser#initialize#3480

Merged
kddnewton merged 1 commit intoruby:mainfrom
koic:resolve_conflict_for_prism_translation_parser_initialize
Feb 25, 2025
Merged

Restore a comment for Prism::Translation::Parser#initialize#3480
kddnewton merged 1 commit intoruby:mainfrom
koic:resolve_conflict_for_prism_translation_parser_initialize

Conversation

@koic
Copy link
Contributor

@koic koic commented Feb 25, 2025

This restores the missing method comments in #3479.

@koic koic force-pushed the resolve_conflict_for_prism_translation_parser_initialize branch from 0eda078 to 7c82ae5 Compare February 25, 2025 17:32
@koic koic changed the title Resolve conflict for Prism::Translation::Parser#initialize Restore a comment for Prism::Translation::Parser#initialize Feb 25, 2025

Racc_debug_parser = false # :nodoc:

# The `builder` argument is used to create the parser using out custom builder class by default.
Copy link
Collaborator

Choose a reason for hiding this comment

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

Should this be:

Suggested change
# The `builder` argument is used to create the parser using out custom builder class by default.
# The `builder` argument is used to create the parser using our custom builder class by default.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oops! I've fixed this typo. Thank you, Ufuk!

This restores the missing method comments in ruby#3479.
@koic koic force-pushed the resolve_conflict_for_prism_translation_parser_initialize branch from 7c82ae5 to 78b8f67 Compare February 25, 2025 18:51
@kddnewton kddnewton merged commit ad7d82c into ruby:main Feb 25, 2025
59 checks passed
@koic koic deleted the resolve_conflict_for_prism_translation_parser_initialize branch February 25, 2025 23:48
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