Skip to content

Conversation

@danieldk
Copy link
Member

No description provided.

@HuggingFaceDocBuilderDev

The docs for this PR live here. All of your documentation changes will be reflected on that endpoint. The docs are available until 30 days after the last update.

MekkCyber
MekkCyber previously approved these changes Nov 18, 2025
Copy link
Collaborator

@MekkCyber MekkCyber left a comment

Choose a reason for hiding this comment

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

Very nice!

Comment on lines +10 to +14
if must_raise:
with pytest.raises(
ImportError, match=r"Kernel requires dependency `nvidia-cutlass-dsl`"
):
get_kernel("kernels-test/python-dep")
Copy link
Collaborator

Choose a reason for hiding this comment

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

an other nit : this path will not be tested since we installed nvidia-cutlass-dsl in the runner before running tests

Copy link
Member Author

Choose a reason for hiding this comment

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

We run tests twice, once without nvidia-cutlass-dsl and once with.

Copy link
Collaborator

Choose a reason for hiding this comment

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

ah sorry yes didn't see that

@danieldk danieldk merged commit 72a1d91 into main Nov 18, 2025
13 checks passed
@danieldk danieldk deleted the validate-dependencies branch November 18, 2025 12:37
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