Remove IDE/user configuration specific stuff #10
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I removed all the user configuration specific stuff, because every time a users chooses a different IDE other than Idea, they have to exclude their IDE dependent files exclusively.
But the bigger issue I think is the IDE configuration. Some users could have a different project configuration and then these changes will be automatically included. This also applies your test server setup. In my case I created a shared test server folder. Easier to update and can be automated too. But then there are other configurations like a activating bungeecord or not too. So I think it's better to exclude this folder too.
This should also make the repo smaller for future commits and additions, because there shouldn't be any binary files except the gradle wrapper.