961191 - New Jetpack CLI (JPM) JEP

Status: RESOLVED FIXED

(Add-on SDK Graveyard :: General, defect, P1)

Comment 1

[Jordan Santell [:jsantell] [@jsantell]]

Attachment: JPM JEP

Comment 2

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

Comment on attachment 8363200 [details] [review]
JPM JEP

Looks good to me.

Comment 3

[Erik Vold [:erikvold] (please needinfo? me)]

Comment on attachment 8363200 [details] [review]
JPM JEP

Looks good to me!  Although maybe we should mention what to do with the experimental options.

Comment 4

[Jordan Santell [:jsantell] [@jsantell]]

Regarding the experimental options currently in CFX, all are listed in the JEP (and categorized by implement, eventually implement, and should not implement)

Comment 5

[Jordan Santell [:jsantell] [@jsantell]]

Merged current feedback into JEP so far

Comment 6

[Irakli Gozalishvili [:irakli] [:gozala] [@gozala]]

Comment on attachment 8363200 [details] [review]
JPM JEP

Putting f- just to indicate that we need to update this pull by reducing
features.

Comment 7

[Erik Vold [:erikvold] (please needinfo? me)]

(In reply to Irakli Gozalishvili [:irakli] [:gozala] [@gozala] from comment #6)
> Comment on attachment 8363200 [details] [review]
> JPM JEP
> 
> Putting f- just to indicate that we need to update this pull by reducing
> features.

I just read over the comments in this pull request and I can't figure out why you f- this Irakli.  Can we get this JEP finished so that we can release this?

Comment 8

[Irakli Gozalishvili [:irakli] [:gozala] [@gozala]]

I believe that either Erik or Jordan are going to go through the pull request to address or object to some of the comments in there. Once that's done feel free to re-request feedback.

Comment 9

[Erik Vold [:erikvold] (please needinfo? me)]

So in the meeting yesterday I believe Irakli said this was done, but it does not appear to have been merged yet.

Comment 10

[Jordan Santell [:jsantell] [@jsantell]]

This is now merged; closing