Skip to content

Move JLD2.jl dependency to a package extension#43

Merged
simsurace merged 2 commits into
masterfrom
jld2-ext
Nov 2, 2025
Merged

Move JLD2.jl dependency to a package extension#43
simsurace merged 2 commits into
masterfrom
jld2-ext

Conversation

@simsurace

Copy link
Copy Markdown
Member

No description provided.

@Krastanov Krastanov left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thanks for setting this up. I think the info message can be done slightly differently, avoiding invalidation when the extension gets loaded.

Comment thread src/persistence.jl Outdated
The latter will lead to invalidation when the extension is loaded
@simsurace simsurace requested a review from Krastanov November 1, 2025 10:30

@Krastanov Krastanov left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

Thanks for taking care of this!

@simsurace simsurace merged commit c996206 into master Nov 2, 2025
2 checks passed
@simsurace simsurace deleted the jld2-ext branch November 2, 2025 20:45
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