Skip to content
This repository was archived by the owner on Apr 6, 2026. It is now read-only.

Add ci output for generated flakes#254

Merged
danieldk merged 1 commit into
mainfrom
ci-output
Oct 10, 2025
Merged

Add ci output for generated flakes#254
danieldk merged 1 commit into
mainfrom
ci-output

Conversation

@danieldk
Copy link
Copy Markdown
Member

This output is like bundle, but only builds one variant for each framework.

Comment thread lib/build.nix
});

# Build multiple Torch extensions.
buildNixTorchExtensions =
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Removed this function, we do not use it anywhere.

Comment thread lib/build.nix
builtins.listToAttrs (lib.map (extensionForTorch { inherit path rev; }) applicableBuildSets');

buildTorchExtensionBundle =
mkTorchExtensionBundle =
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I think the mk naming pattern is a bit nicer, so renamed everything here.

This output is like `bundle`, but only builds one variant for each
framework.
Copy link
Copy Markdown
Contributor

@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.

lgtm

@danieldk danieldk merged commit 0cf2feb into main Oct 10, 2025
28 checks passed
@danieldk danieldk deleted the ci-output branch October 10, 2025 11:56
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants