-
Notifications
You must be signed in to change notification settings - Fork 39
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
Any interest in contributing this to eclipse foundation for use in base Eclipse IDE ? #37
Comments
not to thread jack, but where can one find this json editor for mars? |
What would be required to contributing this to the Eclipse foundation?
|
@jgangemi it is in Eclipse WTP for Neon (https://bugs.eclipse.org/bugs/show_bug.cgi?id=471820) @oyse basically, find which project at eclipse it makes sense to have it in - my first suggestion would be in WTP, you sign the CLA for Eclipse (it is a click of a button), then you open a bug outlining the contribution to see if interest, submit it as a gerrit patch set and we open a so called CQ to get it cleared by Eclipse legal team. You only have 4 people listed on contributor list thus assuming they can be identified as having donated the work under EPL I believe all should be fine for the main source code. Its only if there should be problems with any of your third party dependencies it might be tricky - but snakeyml is under ASL so assuming they are good we should be fine there too. The biggest hurdle is to make sure it isn't just dumped and everyone leaving the project unmaintained afterwards ;) |
@oyse I'd like to add more concrete technical actions, additionally to what @maxandersen says. To get started, it's just a matter of pushing a patch and make it "good enough", then all the IP stuff and other will be taken care of by other contributors.
Then a conversation will start with the project maintainers. Please share the URL of the review on this GitHub issue when Gerrit review is created so other interested parties can get into the conversation to provide more guidance. Thanks in advance! |
And it is already in orbit as well. On Mon, Mar 14, 2016 at 3:49 AM, Mickael Istria notifications@github.com
|
Some questions:
|
Question from someone already consuming YEdit as a dependency (at Pivotal we are using it in STS to support spring-boot yaml properties editor and cloudfoundry manifest.yml editor). How will this rebundling YEdit into WTP affect existing consumers? I hope care will be taken not to make installing WTP a requirement to consume YEdit as dependency. I hope it can retain its 'identity' as a separate bundle with no dependencies on WTP infrastructure (even if it becomes 'officially' a part of WTP). The editor is useful all by itself and really not specific to 'web tools'. Many formats are based on yml and not all of them are 'webby' things. Forcing people to install WTP to get the editor would not be a good thing I.M.O. To be honest I'd be more comfortable if Yedit didn't become part of the 'WTP beast' and remained a small independent project. |
On the positive side... having YEdit be under eclipse umbrella somehow, would help it become more 'mature' and easier to consume for us in some ways. (E.g. the fact is that right now we build our own version of YEdit because otherwise the artifacts are not signed properly). Eclipse build infrastucture will fix those kinds of issues. |
@kdvolder WTP already has plenty of small independent separate installable projects. We could still expose an eclipse marketplace entry for it. About releases - then if you are part of eclipse release train you ship every 2-3 months into the release train. If you want to do builds more often you can do that - it just won't be in the major release train - but this is something we are working on improving, or at least make an option for projects. How often do you want to release ? looking at release tags it seems you are doing about the same. Note, WTP is just one possible place - its just my preference since then we don't need to setup a new build process or release engineering - it just takes part of existing 'machinery' so to speak. |
@kdvolder btw. we are also consuming yedit, actually currently through our usage of springIDE - we would prefer to get it from an actual shared install rather than pivotal custom built one :) And if yedit was in WTP (or at least eclipse foundation release train) it would be possible from cloudfoundry tooling to provide a yaml editor. We got similar interest - OpenShift/Kubernetes supports both json and yaml support - and again, springIDE has a different build than the one from yedit updatesite...would be great if we got that into a more consumable/stable place. Win-win for everybody I think! |
WTP is traditionally on 1 feature + 2 maintenance yearly release. However, that's something that can be adapted: we could make some WTP components have their dedicated releases schedule and site.
WebTools uses Gerrit (which is awesome once you tried it, but it requires some learning to do it well; compared to GitHub pull request that require low learning - to usually do it wrong ;).
Those are very high, and that's how Eclipse IDE is still there and sustainable. Most of the quality level simply comes by the diversity of the community: more people with different goals/scopes/cultures will follow the development and speak up if there is something that is not good in their standard of quality. The simple fact the people speak up improves things. Also, instead of getting into WTP you can consider bootstrapping a new project, independent from WebTools, to get more freedom about infrastructure (Eclipse.org allow projects to be on GitHub and use PRs) and release schedule. But creating a new project requires more effort: there is a proposal to write. I guess people in this conversation would be ready to help and to take a share of the work if you prefer that way. |
Thanks for your great input. I will start the process and then we can take it from there. @kdvolder I share your concern with regards to depending on YEdit means that you need to depend on a big project like WTP, but it seams like this is solvable so that YEdit can become a separate dependency. I think that should be goal in this process. |
I have started the process now and lets see how it goes. I have uploaded the changes as a draft for the moment: https://git.eclipse.org/r/#/c/69787/ Let me know if there are any problems with getting access to it. I guess I probably have to add you as reviewers. I have not uploaded the feature or site part of repository as they might be simpler to just do from scratch. There are also some things we need to take care of in terms of licensning. Most of the code is written by me, but not every single line. This means that the current change actually does not conform to the Eclipse CLA as I can't sign of on other peoples code. We need to make sure that this is taken care of in a good way so that we do not do quite a bit of work and then run into legal trouble in the end. |
Wow, thanks a lot! |
I have published the changes now. Can you check if you have access? The releated bug can be found here: https://bugs.eclipse.org/bugs/show_bug.cgi?id=491050 Feel free send out an announcment on the mailing list. |
Hey, any interest in contributing this to eclipse foundation so we can use it in base Eclipse IDE ?
We got a new json editor for Mars - would be great to see a yaml editor too.
The text was updated successfully, but these errors were encountered: