Skip to content

Patch conda CI logic - #58

Merged
akhanf merged 3 commits into
masterfrom
djay/fix-conda-ci-logic
Feb 14, 2026
Merged

akhanf merged 3 commits into
masterfrom
djay/fix-conda-ci-logic

Conversation

@Dhananjhay

@Dhananjhay Dhananjhay commented Feb 12, 2026 •

Copy link
Copy Markdown
Contributor

Description

This PR addresses the incorrect CI logic for building and pushing cfmm2tar conda package. Now, instead of pushing on every commit to the master branch, it'll build and push a new conda package on manual triggers and tag releases.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update
  • Code quality improvement

Checklist

  • My code follows the code style of this project (ruff check passes)
  • My code is properly formatted (ruff format passes)
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • I have updated the documentation accordingly
  • My changes generate no new warnings

Testing

Manual testing on this branch

@Dhananjhay

Copy link
Copy Markdown
Contributor Author

This is good to merge -- I tested it out via manual trigger and new tag creation!

@akhanf

akhanf commented Feb 14, 2026

Copy link
Copy Markdown
Member

Thanks!

@akhanf
akhanf merged commit b4546e9 into master Feb 14, 2026
4 of 5 checks passed
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