singa-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From GitBox <...@apache.org>
Subject [GitHub] [singa] moazreyad commented on pull request #795: Update the Github Workflow for uploading conda packages
Date Tue, 15 Sep 2020 16:15:42 GMT

moazreyad commented on pull request #795:
URL: https://github.com/apache/singa/pull/795#issuecomment-692821265


   I think it will be better to create a new separate workflow for publishing conda packages.
In the conda package workflow, the build should not use ENABLE_TEST unless it is building
a dev package for developers not a normal package for users. This will make the built code
faster because it can use compiler optimizations which are not available in test mode and
smaller because it does not contain the debug information. 
   
   The packaging workflow (for conda, pypi, debian, docker, etc) should not contain the code
coverage steps because (1) the code is usually not built with test enabled. (2) there is no
need to upload a new coverage report for the same python test cases unless there are special
test cases for the the package process itself.
   
   The packaging workflow may not run with each pull request or merge on the dev branch, but
they can run periodically every week for example or only when merging with the master branch.


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