Skip to content

Conversation

@mr-c
Copy link
Member

@mr-c mr-c commented Aug 18, 2021

No description provided.

@tom-tan
Copy link
Member

tom-tan commented Aug 18, 2021

Great!
It would be nice if README mentions the new container image.

@codecov
Copy link

codecov bot commented Aug 18, 2021

Codecov Report

Merging #1485 (b1016ea) into main (3cf62d8) will decrease coverage by 0.02%.
The diff coverage is n/a.

❗ Current head b1016ea differs from pull request most recent head 74994e5. Consider uploading reports for the commit 74994e5 to get more accurate results
Impacted file tree graph

@@            Coverage Diff             @@
##             main    #1485      +/-   ##
==========================================
- Coverage   65.84%   65.81%   -0.03%     
==========================================
  Files          89       89              
  Lines       15624    15624              
  Branches     3935     3935              
==========================================
- Hits        10287    10283       -4     
- Misses       4249     4253       +4     
  Partials     1088     1088              
Impacted Files Coverage Δ
job.py 61.41% <0.00%> (-0.81%) ⬇️

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 3cf62d8...74994e5. Read the comment docs.

Use non-MUSL based container image so we can benefit from manylinux1
wheels
@mr-c mr-c force-pushed the publish_quay.io branch from b1016ea to c0687a3 Compare August 18, 2021 15:56
@mr-c mr-c force-pushed the publish_quay.io branch from c0687a3 to 74994e5 Compare August 18, 2021 15:59
@mr-c mr-c merged commit 1953cf6 into main Aug 18, 2021
@mr-c mr-c deleted the publish_quay.io branch August 18, 2021 16:00
@mr-c mr-c restored the publish_quay.io branch September 4, 2021 13:22
@mr-c mr-c deleted the publish_quay.io branch September 4, 2021 13:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants