8000 Add EditorConfig file by mrnoname1000 · Pull Request #8657 · jruby/jruby · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Add EditorConfig file #8657

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Conversation

mrnoname1000
Copy link
Contributor

I finally got tired of accidentally indenting the launcher with tabs 😅

This PR adds an EditorConfig file for standardized code style settings across editors. It comes included recent versions of Vim/Neovim, and IntelliJ since about 2014. Not sure what the desired style settings would be for *.rb or *.java but I can add those if desired.

@headius
Copy link
Member
headius commented Feb 25, 2025

I would also like to commit some code standards to the repo (we still get folks occasionally submitting patches with 2-space or 1-tab indents). However this format seems sort of dead? I went to the repository for it and there's hardly any commits in the last five years.

@mrnoname1000
Copy link
Contributor Author

Which repo did you go to? The specification repo has had several updates throughout the past year. Check out their other repositories too, the library and plugin repos seem well maintained. They even have a maven plugin which JRuby could use if all the source was already conformant.

@headius
Copy link
Member
headius commented Feb 26, 2025

Ok that is more promising. I just searched for "editorconfig" and eventually ended up at https://github.com/editorconfig/editorconfig.

@enebo I have no objections to including this. Even IntelliJ recognizes it.

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.

2 participants
0