[Fuego] [PATCH 25/30] ftc: put a fixthis for the documentation
Tim.Bird at sony.com
Tim.Bird at sony.com
Mon Jun 11 20:50:03 UTC 2018
> -----Original Message-----
> From: Daniel
> > -----Original Message-----
> > From: Tim.Bird at sony.com [mailto:Tim.Bird at sony.com]
> > > + This interface may change in the future.
> > > + FIXTHIS: add documentation about timeout,reboot,rebuild"""),
> >
> > OK - I don't like this very much. This is not a normal FIXTHIS, in
> > that it's visible to end users. Most FIXTHIS comments are
> > comments inside the code. This one is, IMHO, very high priority.
> > Other FIXTHIS items range from low priority feature suggestions,
> > to "should check this some time" items.
> >
> > User's shouldn't see notes about the state of the code when they
> > are looking for help.
> >
> > Can you work on this soon? I don't want this one to go out in a
> > release. I'm considering re-wording it "MUSTFIX:", and having a
> > regression check for such terms in my release test.
> >
> > Let me know what you think.
>
> OK, I fixed this. I will send the patches on top of your tree once you have
> merged them.
OK - sounds good.
-- Tim
>
> > > "rm-jobs": ("Removes jobs from Jenkins.",
> > > """Usage: ftc rm-jobs [--remove-logs]
> > > <board>.<testspec>.<testtype>.<testcase>
> > > @@ -239,6 +240,9 @@ would run the pre_test, pre_check, build and
> deploy
> > > phases of the test.
> > > This is useful during development of a test (ie. for testing tests).
> > > Use caution running later phases of a test without their normal
> > > precursors (e.g. run, without build or deploy).
> > > +
> > > +FIXTHIS: add documentation about timeout,reboot,rebuild,dynamic-
> vars
> > Same issue as above.
> >
> > > +
> > > """),
> > >
> > > "build-jobs": ("Build one or more jobs (to execute tests) in Jenkins.",
> > > --
> > > 2.7.4
> >
> >
> > Applied - but changed to MUSTFIX.
> > -- Tim
> >
>
>
>
More information about the Fuego
mailing list