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

JBIDE-17733: Remove EGit from Central discovery #182

Merged
merged 1 commit into from Jul 3, 2014
Merged

JBIDE-17733: Remove EGit from Central discovery #182

merged 1 commit into from Jul 3, 2014

Conversation

mickaelistria
Copy link
Contributor

No description provided.

@nickboldt
Copy link
Member

Proved that both Eclipse platform and Eclipse Standard bundle both permit egit and jgit to be installed (or include it so it's preinstalled), so yes, we probably don't need this in Central. But does it hurt? We have other stuff in Central that is preinstalled in JBDS, like the jst.server connector.

So... +0, remove it if you need to, but seems not necessary.

@mickaelistria
Copy link
Contributor Author

On 07/03/2014 12:07 AM, Nick Boldt wrote:

Proved that both Eclipse platform and Eclipse Standard bundle both
permit egit and jgit to be installed (or include it so it's
preinstalled), so yes, we probably don't need this in Central. But
does it hurt?
We have other stuff in Central that is preinstalled in JBDS, like the
jst.server connector.

Too much useless UI reduces the visibity of the useful parts of UI.
I suggest you add a comment about other useless Central features in
JBIDE-17733 and that we generalize the issue to "Cleanup useless
features from Central"

So... +0, remove it if you need to, but seems not necessary.
Ok, let's delay JBIDE-17733 to CR1.

Mickael Istria
Eclipse developer at JBoss, by Red Hat http://www.jboss.org/tools
My blog http://mickaelistria.wordpress.com - My Tweets
http://twitter.com/mickaelistria

@mickaelistria mickaelistria merged commit e6f42c3 into jbosstools:master Jul 3, 2014
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