You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@plc4x.apache.org by GitBox <gi...@apache.org> on 2018/01/09 14:05:46 UTC

[GitHub] dlaboss commented on issue #1: Warning cleanup

dlaboss commented on issue #1: Warning cleanup
URL: https://github.com/apache/incubator-plc4x/pull/1#issuecomment-356292913
 
 
   Hi Justin, I was expecting to merge it myself, but thanks :-)
   Isn't is just better to consistently work in the github fork/feature-branch/PR flow so as to reinforce its use?  It just makes it easier for folks to identify/review/comment on changes. And when build/test checks are integrated into the PR flow their "passing" is made readily apparent in the PR.

----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on 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


With regards,
Apache Git Services