Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Rename DeadEnd -> SyntaxSuggest #154

Merged
merged 1 commit into from
Jul 26, 2022
Merged

Rename DeadEnd -> SyntaxSuggest #154

merged 1 commit into from
Jul 26, 2022

Conversation

schneems
Copy link
Collaborator

Based on this conversation (https://bugs.ruby-lang.org/issues/18159#note-29) I was asked to change the name of DeadEnd because it might sound scary. This name is more meaningful as it represents what it does. It suggests the location of syntax errors.

@schneems schneems force-pushed the schneems/syntax_suggest branch 2 times, most recently from e05b789 to a7f34da Compare July 26, 2022 19:05
Based on this conversation (https://bugs.ruby-lang.org/issues/18159#note-29) I was asked to change the name of DeadEnd because it might sound scary. This name is more meaningful as it represents what it does. It suggests the location of syntax errors.
@schneems schneems merged commit 3e4e940 into main Jul 26, 2022
@schneems schneems deleted the schneems/syntax_suggest branch July 26, 2022 19:30
@hsbt
Copy link
Member

hsbt commented Jul 26, 2022

@schneems Huge thanks for your thoughts and work ❤️

@AlexWayfer
Copy link

It worthed to be mentioned in the README, I guess. It's quite strange to watch for a project, or maybe even search for it by old memories, and found nothing, but very similar with very different name without mentions of renaming.

Also I remind about VS Code plugins.

@schneems
Copy link
Collaborator Author

schneems commented Aug 4, 2022

@AlexWayfer hope this is helpful a30db92.

Also, @mauro-oto, I'm not sure how to move forward with the vscode plugin if we change the CLI name.

@AlexWayfer
Copy link

@schneems I think it's OK, anyway better than nothing, thank you.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants