Skip to content

Conversation

vtjnash
Copy link
Member

@vtjnash vtjnash commented May 30, 2025

Fixes #50985

(I messed up my github fu, so this is a new PR, identical to the old one)

@vtjnash vtjnash added merge me PR is reviewed. Merge when all tests are passing backport 1.12 Change should be backported to release-1.12 labels May 30, 2025
@JeffBezanson JeffBezanson merged commit aa06976 into master May 30, 2025
10 checks passed
@JeffBezanson JeffBezanson deleted the jn/50985 branch May 30, 2025 18:07
@topolarity topolarity removed the merge me PR is reviewed. Merge when all tests are passing label May 30, 2025
Comment on lines +32 to +35
elseif a === b
return a
elseif !isa(a, Type) || !isa(b, Type)
return Any
Copy link
Member

Choose a reason for hiding this comment

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

Is it intentional that typejoin(3, 3, 3) returns 3, instead of Any?

Copy link
Member

Choose a reason for hiding this comment

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

These seem a bit inconsistent, I guess:

julia> typejoin(3, 3, 3)
3

julia> typejoin(3, 3, 3.0)
Any

Copy link
Member Author

Choose a reason for hiding this comment

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

Those are both correct

KristofferC pushed a commit that referenced this pull request Jun 2, 2025
Fixes #50985

(cherry picked from commit aa06976)
@KristofferC KristofferC mentioned this pull request Jun 2, 2025
58 tasks
@KristofferC KristofferC removed the backport 1.12 Change should be backported to release-1.12 label Jun 4, 2025
@sloede
Copy link
Contributor

sloede commented Sep 1, 2025

@vtjnash Just confirmed with v1.12-rc0 that the fix works - thanks a lot for taking care of this!

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.

Type instability with Base.promote_type and Static.jl types for Julia v1.10-beta2 and --check-bounds=no
6 participants