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

A first stab at Eclipse support #19

Closed
wants to merge 1 commit into from

Conversation

chids
Copy link
Contributor

@chids chids commented May 10, 2011

I'm way too new to Scala, SBT and their possible integration with Maven and/or Eclipse to be certain that this is the way to go about Eclipse support or that it's even a good idea. It does however generate project files and doesn't seem to cause any trouble. Apply or discard at as you see fit.

@codahale
Copy link
Contributor

I'd recommend checking out sbt-eclipsify instead of baking the Eclipse files into the project. Feel free to reopen with sbt-eclipsify support, though!

@codahale codahale closed this May 10, 2011
@chids
Copy link
Contributor Author

chids commented May 11, 2011

Uhm, did you read the diff? ;)

This pull request is adding sbt-eclipsify support, it's not baking the Eclipse files into the project - even though I can see that my issue description might have given that impression.

@codahale codahale reopened this May 11, 2011
@codahale
Copy link
Contributor

Ah, sorry -- that's what I get for trying to go through pull requests on my phone.

@codahale
Copy link
Contributor

I rebased it, resolved the conflict, and applied the patch. Thanks!

@codahale codahale closed this May 11, 2011
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