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

Import statement order config file for Eclipse #2192

Merged
merged 1 commit into from Feb 29, 2016
Merged

Import statement order config file for Eclipse #2192

merged 1 commit into from Feb 29, 2016

Conversation

rzats
Copy link
Member

@rzats rzats commented Feb 29, 2016

Fixes #2117.

Note: since .importorder is a semantic extension, I've chosen not to name this file at all - looks clean and .gitignore-like. This may cause some awkward issues on Windows, though, so it's subject to change.

Usage:

  • In Eclipse, go to Window → Preferences → Java → Code Style → Organize Imports.
  • Import the import order file, click Apply.

screenshot of Organize Imports menu in Eclipse

  • [Recommended] Switch to Editor → Save Actions, check Perform the selected actions on save and Organize imports.

screenshot of Save Actions menu in Eclipse

@GooeyHub
Copy link
Member

Can one of the admins please verify this patch?

@Cervator
Copy link
Member

ok to test

Hi @rzats and thanks for the PR! :-)

@GooeyHub
Copy link
Member

Refer to this link for build results (access rights to CI server needed):
http://jenkins.terasology.org/job/TerasologyPRs/444/
Hooray Jenkins reported success with all tests good!

@Cervator Cervator added this to the v1.0.0 milestone Feb 29, 2016
@Cervator Cervator merged commit 4dbbec8 into MovingBlocks:develop Feb 29, 2016
@Cervator Cervator added Category: Build/CI Requests, Issues and Changes targeting gradle, groovy, Jenkins, etc. in progress and removed in progress labels Feb 29, 2016
@Cervator
Copy link
Member

Merged! And I copied the instructions to https://github.com/MovingBlocks/Terasology/wiki/Eclipse

I know we have a few other config files like that, feel free to enhance that page with how to use the others. I know @msteiger and a few use Eclipse, but I'm on IntelliJ so not as familiar :-)

We had a few related instructions before but I've been on a bit of a "Heal it with fire!" rampage in there and haven't quite built everything back up yet!

Also added you to the credits, let me know if you're in the forum for a contributor badge :-)

@rzats rzats removed the in progress label Apr 5, 2016
@rzats rzats changed the title Add import statement order config file for Eclipse Import statement order config file for Eclipse Apr 14, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category: Build/CI Requests, Issues and Changes targeting gradle, groovy, Jenkins, etc.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants