nuttx-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [incubator-nuttx] patacongo commented on issue #886: Cygwin sim:nsh failed on apps/builtin
Date Mon, 27 Apr 2020 19:19:42 GMT

patacongo commented on issue #886:
URL: https://github.com/apache/incubator-nuttx/issues/886#issuecomment-620181502


   > I just about have a CI run working for Cygwin but it is ugly and really slow. I'll
put up a Draft PR tonight and we can work on getting the Windows testing in a much better
place.
   
   I think we should be the PR checks short and sweet.  Just enough to give us some confidence
that the change is not generally lethal.  To me this could be primarily Linux tests of various
configurations.  More exhaustive testing should be left for the nightly build.
   
   If a Windows CI is really time consuming too much time for efficent PR checks, perhaps
only a nightly test would be sufficient.  That would not detect breakage on the PR, but would
still catch the breakage post-merge.  Which, at least to me, is and acceptable performance/thoroughness
tradeoff.


----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



Mime
View raw message