yunfengzhou-hub commented on code in PR #149:
URL: https://github.com/apache/flink-ml/pull/149#discussion_r958399107


##########
.github/workflows/python-checks.yml:
##########
@@ -32,6 +32,12 @@ jobs:
       - name: Install dependencies
         working-directory: flink-ml-python/dev
         run: python -m pip install -r dev-requirements.txt
+      - name: Build Python package
+        working-directory: flink-ml-python
+        run: python setup.py sdist
+      - name: Install Python package
+        working-directory: flink-ml-python
+        run: python -m pip install dist/*.tar.gz

Review Comment:
   I have added the license header to Github Action config files.
   
   About creating a workflow to build the python package, I think there is no 
need to do so now, given that the workload to build the package on a local 
machine is acceptable so far, and the process is not complicated. 



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

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to