Skip to content

Fix circular import#812

Merged
tchaton merged 1 commit into
Lightning-AI:mainfrom
vini-fda:fix/circular-import
Apr 28, 2026
Merged

Fix circular import#812
tchaton merged 1 commit into
Lightning-AI:mainfrom
vini-fda:fix/circular-import

Conversation

@vini-fda
Copy link
Copy Markdown
Contributor

Before submitting
  • Was this discussed/agreed via a Github issue? (no need for typos and docs improvements)
  • Did you read the contributor guideline, Pull Request section?
  • Did you make sure to update the docs?
  • Did you write any new necessary tests?

What does this PR do?

Fixes #811.

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 24, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 81%. Comparing base (1fdfad7) to head (8f02915).

Additional details and impacted files
@@         Coverage Diff         @@
##           main   #812   +/-   ##
===================================
  Coverage    81%    81%           
===================================
  Files        54     54           
  Lines      7617   7617           
===================================
  Hits       6143   6143           
  Misses     1474   1474           
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Copy Markdown
Collaborator

@deependujha deependujha left a comment

Choose a reason for hiding this comment

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

Thanks for the fix :)

@tchaton tchaton merged commit d99a035 into Lightning-AI:main Apr 28, 2026
48 of 51 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.

Circular import in commands.py

4 participants