Skip to content

Conversation

@zhenrongliew
Copy link
Contributor

Fixes bug in issue #549: circular import error when importing kirin.dialects.py.

Small edits to src/kirin/dialects/func/closurefield.py and src/kirin/dialects/func/lambdalifting.py.
Delaying import.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 28, 2025

PR Preview Action v1.6.2
Preview removed because the pull request was closed.
2025-10-29 18:01 UTC

@codecov
Copy link

codecov bot commented Oct 28, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@github-actions
Copy link
Contributor

github-actions bot commented Oct 28, 2025

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
11464 10204 89% 0% 🟢

New Files

File Coverage Status
src/kirin/dialects/func/rewrite/_init_.py 100% 🟢
TOTAL 100% 🟢

Modified Files

File Coverage Status
src/kirin/dialects/func/_init_.py 100% 🟢
TOTAL 100% 🟢

updated for commit: 6a1fe30 by action🐍

Copy link
Collaborator

@kaihsin kaihsin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can you put rewrite under dialects/func/rewrite, similar to dialects/ilist ?

@zhenrongliew
Copy link
Contributor Author

can you put rewrite under dialects/func/rewrite, similar to dialects/ilist ?

Moved. Updated init.py and tests.

@zhenrongliew zhenrongliew requested a review from kaihsin October 28, 2025 17:24
@Roger-luo Roger-luo merged commit f5bee94 into main Oct 29, 2025
13 checks passed
@Roger-luo Roger-luo deleted the dl/circular-import-lambdalifting branch October 29, 2025 18:01
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.

4 participants