Skip to content

Conversation

@kshyatt
Copy link
Member

@kshyatt kshyatt commented Jan 27, 2026

No description provided.

@kshyatt kshyatt requested a review from lkdvos January 27, 2026 11:19
@codecov
Copy link

codecov bot commented Jan 27, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
see 1 file with indirect coverage changes

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

Copy link
Member

@lkdvos lkdvos left a comment

Choose a reason for hiding this comment

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

I think the Mooncake changes look great, but I don't like the JET changes. Can we report an issue to JET for this, and just leave the rest as-is for now?

@kshyatt
Copy link
Member Author

kshyatt commented Jan 27, 2026

The problem is our codequality tests will fail, unless you want me to add the disabler comments in these spots?

@github-actions
Copy link

github-actions bot commented Jan 28, 2026

Your PR no longer requires formatting changes. Thank you for your contribution!

@lkdvos
Copy link
Member

lkdvos commented Jan 28, 2026

Linking this issue here: aviatesk/JET.jl#801

@kshyatt kshyatt changed the title Bump Mooncake and JET Bump Mooncake Jan 29, 2026
@kshyatt
Copy link
Member Author

kshyatt commented Jan 29, 2026

Let's see if we can just bump Mooncake only

@kshyatt kshyatt enabled auto-merge (squash) January 29, 2026 13:47
@lkdvos lkdvos disabled auto-merge January 29, 2026 14:56
@lkdvos lkdvos merged commit ad0490c into main Jan 29, 2026
10 checks passed
@lkdvos lkdvos deleted the ksh/bumps branch January 29, 2026 14:58
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.

3 participants