Skip to content

Fixes to the CD Pipeline #334

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Fixes to the CD Pipeline #334

wants to merge 4 commits into from

Conversation

f-allian
Copy link
Contributor

@f-allian f-allian commented Jun 30, 2025

Previously the CD pipeline was incorrectly configured with the following steps:

Solution:

  • Removed installation from source. The pipeline now just builds and uploads the distributions directly (see image below that causal_testing.testing.causal_effect.py is now correctly included in the build process.)
  • This simplifies the setup process and ensures all tools are up-to-date.
  • Minor: also made /estimation directory a module as this was assumed due to the implicit namespace conventions in Python 3+.
Screenshot 2025-06-30 at 14 06 14

@f-allian f-allian self-assigned this Jun 30, 2025
@f-allian f-allian added bug Something isn't working enhancement New feature or request labels Jun 30, 2025
Copy link

github-actions bot commented Jun 30, 2025

🦙 MegaLinter status: ✅ SUCCESS

Descriptor Linter Files Fixed Errors Elapsed time
✅ PYTHON black 32 0 1.0s
✅ PYTHON pylint 32 0 6.62s

See detailed report in MegaLinter reports

MegaLinter is graciously provided by OX Security

Copy link

codecov bot commented Jun 30, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 95.75%. Comparing base (8711cab) to head (febf7e7).
Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #334   +/-   ##
=======================================
  Coverage   95.75%   95.75%           
=======================================
  Files          27       27           
  Lines        1624     1624           
=======================================
  Hits         1555     1555           
  Misses         69       69           

Continue to review full report in Codecov by Sentry.

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@f-allian f-allian marked this pull request as ready for review June 30, 2025 13:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants