Skip to content

Switch CI to Github Action with full x86 matrix build#39

Merged
lamhoangtung merged 35 commits intostarlletefrom
ci_cd_py38_py_39_arm
Apr 15, 2021
Merged

Switch CI to Github Action with full x86 matrix build#39
lamhoangtung merged 35 commits intostarlletefrom
ci_cd_py38_py_39_arm

Conversation

@lamhoangtung
Copy link
Copy Markdown
Contributor

#38 Switch CI to Github Action with full x86 matrix build. Including:

  • 3 OS: Windows, Mac, Linux
  • 4 Python version: 3.6, 3.7, 3.8, 3.9

@lamhoangtung lamhoangtung added the enhancement New feature or request label Apr 14, 2021
@lamhoangtung lamhoangtung self-assigned this Apr 14, 2021
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 14, 2021

Codecov Report

Merging #39 (85dbe8c) into starllete (4427f3f) will decrease coverage by 2.21%.
The diff coverage is 40.00%.

Impacted file tree graph

@@              Coverage Diff              @@
##           starllete      #39      +/-   ##
=============================================
- Coverage      56.40%   54.19%   -2.22%     
=============================================
  Files             47       46       -1     
  Lines           4005     4021      +16     
=============================================
- Hits            2259     2179      -80     
- Misses          1746     1842      +96     
Impacted Files Coverage Δ
mlchain/workflows/background.py 81.52% <40.00%> (-5.84%) ⬇️
mlchain/workflows/pipeline.py 19.20% <0.00%> (-64.41%) ⬇️
mlchain/workflows/task.py 67.27% <0.00%> (-12.73%) ⬇️
mlchain/base/log.py 67.85% <0.00%> (-12.15%) ⬇️
mlchain/cli/mlchain_server.py
mlchain/cli/run.py 62.16% <0.00%> (+0.12%) ⬆️
mlchain/server/flask_server.py 60.24% <0.00%> (+0.16%) ⬆️
mlchain/workflows/limiter.py 76.11% <0.00%> (+0.36%) ⬆️
mlchain/storage/object_storage.py 13.79% <0.00%> (+0.37%) ⬆️
... and 4 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4427f3f...85dbe8c. Read the comment docs.

@lamhoangtung lamhoangtung marked this pull request as draft April 15, 2021 03:29
@lamhoangtung lamhoangtung marked this pull request as ready for review April 15, 2021 08:48
@lamhoangtung
Copy link
Copy Markdown
Contributor Author

Looks like mlchain.workflows.Pipeline fail randomly on MacOS 3.8+. Will need to fix this in the future. Proceed to merge for now.

@lamhoangtung lamhoangtung merged commit be90c81 into starllete Apr 15, 2021
@lamhoangtung lamhoangtung deleted the ci_cd_py38_py_39_arm branch April 15, 2021 13:39
@lamhoangtung lamhoangtung linked an issue May 17, 2021 that may be closed by this pull request
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support full CI matrix build for x86

1 participant