Skip to main content

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index] [List Home]
Re: [wtp-dev] JSON Editor contribution is approved

Hi, Mickael!

Yes, that's exactly what we going to do.

Victor Rubezhny

On 11/25/2015 10:00 AM, Mickael Istria wrote:
Good news.
Here are my 2 cents.

On 11/25/2015 12:31 AM, Victor V. Rubezhny wrote:
During few discussions that took place during last few weeks we're all agreed to put this contribution into WTP SSE repository (correct me if I'm wrong), so given the fact of moving the repository into Gerrit and merging the webtools.sourceediting.tests repository into it, I have the following questions:

- Should we wait for the finish of Gerrit adoption process or we can attach the final patch for the issue (bug #471820) before we proceed with actual code pushing? And how the adoption goes for webtools.sourceediting? (IMHO, it's better to propose a change in Gerrit instead of just a text patch attachment for the contribution...)
- Should we wait for the webtools.sourceediting.tests to be finally merged into webtools.sourceediting repository? (As far as I understood, not only the tests are to be merged, but the overall maven project is to be fixed during the merge, so we have to use the latest projects configuration for the contribution.)
I would say that we already waited too much. There are already 2 things we're waiting on for this repository (Gerrit, merge), let's avoid adding one more.
If you want a review mechanism, you can consider turning the contribution into a commit on a branch at GitHub and review/amed the commit there until it's ready to get merged.

Mickael Istria
Eclipse developer at JBoss, by Red Hat
My blog - My Tweets

wtp-dev mailing list
To change your delivery options, retrieve your password, or unsubscribe from this list, visit

CONFIDENTIALITY NOTICE: This email and files attached to it are confidential. If you are not the intended recipient you are hereby notified that using, copying, distributing or taking any action in reliance on the contents of this information is strictly prohibited. If you have received this email in error please notify the sender and delete this email.

Back to the top