You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@nuttx.apache.org by GitBox <gi...@apache.org> on 2021/05/07 17:22:14 UTC

[GitHub] [incubator-nuttx] gustavonihei opened a new pull request #3676: tools/nxstyle: Extend check to any URI scheme

gustavonihei opened a new pull request #3676:
URL: https://github.com/apache/incubator-nuttx/pull/3676


   ## Summary
   This PR intends to extend the **nxstyle** HTTP and HTTPS schemes checking to any string that matches the pattern "://".
   
   It is a proposal for fixing the false-positive from https://github.com/apache/incubator-nuttx-apps/pull/705.
   
   There may be some scenarios that may induce false-negatives, but I believe that will not be much frequent.
   
   ## Impact
   No more false-positive reports from nxstyle on schemes other than HTTP and HTTPS.
   
   ## Testing
   https://github.com/apache/incubator-nuttx-apps/pull/705 successful check.
   


-- 
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

[GitHub] [incubator-nuttx] xiaoxiang781216 merged pull request #3676: tools/nxstyle: Extend check to any URI scheme

Posted by GitBox <gi...@apache.org>.

xiaoxiang781216 merged pull request #3676:
URL: https://github.com/apache/incubator-nuttx/pull/3676


   


-- 
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