Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Prepare v1.82.1 #105

Merged
merged 6 commits into from
Nov 11, 2023
Merged

Prepare v1.82.1 #105

merged 6 commits into from
Nov 11, 2023

Conversation

JamesWrigley
Copy link
Member

Copied from #104:
This is a patch release that fixes compat bounds for some dependencies, which were previously allowing newer-than-actually-compatible versions. See: #100 (comment)

The main change here is to remove the unions from the bounds of ImGuiGLFWBackend.jl, ImGuiOpenGLBackend.jl, and LibCImGui.jl. e.g. ~1.82, 1 for LibCImGui.jl means that the allowed versions would be [1.82 - 1.83) union [1 - 2) (if I read the docs correctly), which is not correct since LibCImGui.jl 1.89 is breaking but would still be allowed by that second bound.

@JamesWrigley JamesWrigley self-assigned this Nov 10, 2023
@JamesWrigley JamesWrigley mentioned this pull request Nov 10, 2023
IanButterworth and others added 6 commits November 10, 2023 23:57
This is a patch release that fixes compat bounds for some dependencies, which
were previously allowing newer-than-actually-compatible versions.
@JamesWrigley
Copy link
Member Author

(also took the liberty of backporting some docs fixes from @IanButterworth ❤️ )

@JamesWrigley JamesWrigley requested a review from Gnimuc November 10, 2023 23:03
@Gnimuc
Copy link
Member

Gnimuc commented Nov 11, 2023

Thanks for working on this!

@JamesWrigley JamesWrigley merged commit a65163a into release-1.82 Nov 11, 2023
12 checks passed
@JamesWrigley JamesWrigley deleted the v1.82.1 branch November 11, 2023 10:43
@JamesWrigley
Copy link
Member Author

NP :) Shall I tag/register the new version?

@Gnimuc
Copy link
Member

Gnimuc commented Nov 11, 2023

Feel free to tag a new version if it's ready.

@JamesWrigley
Copy link
Member Author

Done: JuliaRegistries/General#95174

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.

3 participants