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

languages: Fix ruby rakefile and gemfile file type #2875

Merged

Conversation

kihaya
Copy link
Contributor

@kihaya kihaya commented Jun 25, 2022

I found default listed ruby file type (rakefile and gemfile) does not match Rakefile and Gemfile, so fix them.

languages.toml Outdated
@@ -446,7 +446,7 @@ source = { git = "https://github.com/cstrahan/tree-sitter-nix", rev = "6b71a810c
name = "ruby"
scope = "source.ruby"
injection-regex = "ruby"
file-types = ["rb", "rake", "rakefile", "irb", "gemfile", "gemspec"]
file-types = ["rb", "rake", "Rakefile", "irb", "Gemfile", "gemspec"]
Copy link
Member

Choose a reason for hiding this comment

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

I've seen some foo.rakefile files in the wild. I'm not sure it's technically valid of if it's just allowed by case-insensitive file systems. Let's have both rakefile and Rakefile for anyone using the lowercase version (might as well do the same for the gemfile too)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@the-mikedavis
Thank you for your review.
I changed to both file type is available.

ea7a160

Copy link
Member

@the-mikedavis the-mikedavis left a comment

Choose a reason for hiding this comment

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

Thank you!

@the-mikedavis the-mikedavis merged commit 24351c2 into helix-editor:master Jun 26, 2022
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.

None yet

2 participants