Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

kohlisid
Copy link
Contributor

@kohlisid kohlisid commented Jan 30, 2025

Have added this to unblock the builds currently, the poetry version is pinned in the example docker files which was causing the builds to fail. Hence we need to recreate with poetry lock.

This is temp fix, I will update the Dockerfiles to use an optimized version which I have for other use cases

Signed-off-by: Sidhant Kohli <[email protected]>
@kohlisid kohlisid marked this pull request as draft January 30, 2025 19:58
Copy link

codecov bot commented Jan 30, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 94.13%. Comparing base (cbdec86) to head (c263b77).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #207      +/-   ##
==========================================
- Coverage   94.17%   94.13%   -0.05%     
==========================================
  Files          55       55              
  Lines        2317     2317              
  Branches      119      119              
==========================================
- Hits         2182     2181       -1     
- Misses         98       99       +1     
  Partials       37       37              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Signed-off-by: Sidhant Kohli <[email protected]>
Signed-off-by: Sidhant Kohli <[email protected]>
@kohlisid kohlisid changed the title [DONT MERGE]: test build chore: fix docker builds for examples Jan 31, 2025
@kohlisid kohlisid marked this pull request as ready for review January 31, 2025 01:23
Signed-off-by: Sidhant Kohli <[email protected]>
@vigith vigith merged commit e6e336d into main Feb 1, 2025
11 checks passed
@vigith vigith deleted the test-build branch February 1, 2025 01:56
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.

2 participants