Skip to content

Conversation

@ahal
Copy link
Collaborator

@ahal ahal commented Sep 19, 2025

…true

@ahal ahal self-assigned this Sep 19, 2025
@ahal ahal requested a review from a team as a code owner September 19, 2025 15:20
@ahal ahal requested a review from bhearsum September 19, 2025 15:20
@ahal
Copy link
Collaborator Author

ahal commented Sep 19, 2025

I'm somewhat surprised we never ran into this bug before now, given that unique-kinds defaults to True.

Copy link
Contributor

@bhearsum bhearsum left a comment

Choose a reason for hiding this comment

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

This should be fine; it's a breaking change though AFAICT?

@ahal
Copy link
Collaborator Author

ahal commented Sep 22, 2025

I don't consider it breaking because if you hit this case, the result would be a KeyError. So imo, this is just fixing an exception that can happen in some circumstances.

@ahal ahal merged commit a7d0e0b into taskcluster:main Sep 22, 2025
17 checks passed
@ahal ahal deleted the from_deps_fix branch September 22, 2025 18:41
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.

2 participants