<div dir="ltr">If your&#39;e using container builder, I would call it &quot;container&quot; instead of CDISE.  Even if we switch to initializer or configurator, this should become the container class.<div><br></div><div>I&#39;m in favor of this being its own spec jar, and really anything that is SE specific going there.  If we end up with something that is web specific, we should consider moving that out as well.<br><div><br></div><div>All other issues I see are commented in the PR.  Looks like we&#39;re really close.</div><div><br></div><div>I&#39;m not going to be able to attend tomorrows meeting, unfortunately, but still wanted to see if we could coordinate a real time chat around CDI-30.</div><div><br></div><div>John<br><br><div class="gmail_quote"><div dir="ltr">On Mon, Jun 20, 2016 at 4:48 AM Antoine Sabot-Durand &lt;<a href="mailto:antoine@sabot-durand.net">antoine@sabot-durand.net</a>&gt; wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi guys,<div><br></div><div>I posted modification on PR 290 [1] according to last meeting.</div><div><br></div><div>I still have 2 points that stays open for me (you may have more) :</div><div><br></div><div>- Place and name of the package: should we decide to package SE support in a specific jar, it will be probably better to have a better package name at a higher level</div><div><br></div><div>- Name for CDISE class. Perhaps using Container or SeContainer  would be better. WDYT ?</div><div><br></div><div>Antoine</div><div><br></div><div>[1] <a href="https://github.com/cdi-spec/cdi/pull/290" target="_blank">https://github.com/cdi-spec/cdi/pull/290</a></div></div>
_______________________________________________<br>
cdi-dev mailing list<br>
<a href="mailto:cdi-dev@lists.jboss.org" target="_blank">cdi-dev@lists.jboss.org</a><br>
<a href="https://lists.jboss.org/mailman/listinfo/cdi-dev" rel="noreferrer" target="_blank">https://lists.jboss.org/mailman/listinfo/cdi-dev</a><br>
<br>
Note that for all code provided on this list, the provider licenses the code under the Apache License, Version 2 (<a href="http://www.apache.org/licenses/LICENSE-2.0.html" rel="noreferrer" target="_blank">http://www.apache.org/licenses/LICENSE-2.0.html</a>). For all other ideas provided on this list, the provider waives all patent and other intellectual property rights inherent in such information.</blockquote></div></div></div></div>