Skip to content

refactor: BREAKING: make name optional#4410

Merged
ruben-arts merged 9 commits intoprefix-dev:mainfrom
baszalmstra:remove_name_from_build_types
Aug 26, 2025
Merged

refactor: BREAKING: make name optional#4410
ruben-arts merged 9 commits intoprefix-dev:mainfrom
baszalmstra:remove_name_from_build_types

Conversation

@baszalmstra
Copy link
Contributor

@baszalmstra baszalmstra commented Aug 21, 2025

What

This was required:

[package]
name = "bla"
version = "1.2.3"

[package.build.backend]
name = "pixi-build-ros"
version = "*"

With the current changes in this PR you can make a project build by pixi by only adding the following:

[package.build.backend]
name = "pixi-build-ros"
version = "*"

BREAKING:

This is a breaking change for the lockfiles, as the name in the source hash of the lockfile. Which broke the hash of the lockfiles. This will only be breaking for anyone using the preview = ["pixi-build"]!

Blocks:

@baszalmstra baszalmstra requested a review from ruben-arts August 21, 2025 08:35
Copy link
Contributor

@ruben-arts ruben-arts left a comment

Choose a reason for hiding this comment

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

self review

self.index_json
.as_ref()
.map(|j| j.name.clone())
.unwrap_or_else(|| {
Copy link
Contributor

Choose a reason for hiding this comment

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

Not sure what to do here, we could also make this an unreachable as that would be really weird. But rattler-build currently already errors.

@ruben-arts ruben-arts marked this pull request as ready for review August 26, 2025 14:02
Copy link
Contributor Author

@baszalmstra baszalmstra left a comment

Choose a reason for hiding this comment

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

Comments

Copy link
Contributor

@ruben-arts ruben-arts left a comment

Choose a reason for hiding this comment

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

Self approve after discussion with @baszalmstra

@ruben-arts ruben-arts added the breaking Breaks something in the api or config label Aug 26, 2025
@ruben-arts ruben-arts changed the title refactor: make name optional refactor: BREAKING: make name optional Aug 26, 2025
@ruben-arts ruben-arts enabled auto-merge (squash) August 26, 2025 14:54
@ruben-arts ruben-arts merged commit 15b5574 into prefix-dev:main Aug 26, 2025
52 of 77 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

breaking Breaks something in the api or config

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants