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

Remove IDE-specific files from .gitignore and git #60

Closed
wants to merge 1 commit into from
Closed

Remove IDE-specific files from .gitignore and git #60

wants to merge 1 commit into from

Conversation

krichter722
Copy link

Remove IDE-specific files from .gitignore which are supposed to be managed in .git/info/exclude

They're are supposed to be managed in .git/info/exclude.
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.04%) to 70.124% when pulling 196ae8b on krichter722:gitignore into 367da7d on tools4j:master.

@terzerm
Copy link
Member

terzerm commented Dec 2, 2018

Not sure what problem you are trying to solve. We are happy the way this is currently configured with idea files ignored (except two files) and checked in to .gitignore.

If you describe the problem in a new issue we may decide to come back to this. As it stands I have to reject it as I am not understanding the motivation for it.

@terzerm terzerm closed this Dec 2, 2018
@krichter722
Copy link
Author

The project is more easy to maintain if it depends on the build tool (and eventual plugins) only. This is achieved by keeping all IDE-specific files out of the SCM. Furthermore you keep the project open to developers who don't want to develop with a specific IDE which is unnecessary if it depends on the build-tool only. Codestyle only makes sense to be maintained with static analysis tools like checkstyle, not a formatter (you can offer a file that pleases checkstyle for different IDEs outside the source root). There's currently no problem, but there might be in the future.

@krichter722 krichter722 deleted the gitignore branch October 22, 2021 22:27
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