Skip to content

Fix compatibility with Julia 1.13+ memhash removal#14

Merged
tpapp merged 1 commit intotpapp:masterfrom
vtjnash:fix-memhash-compat
Feb 14, 2026
Merged

Fix compatibility with Julia 1.13+ memhash removal#14
tpapp merged 1 commit intotpapp:masterfrom
vtjnash:fix-memhash-compat

Conversation

@vtjnash
Copy link
Contributor

@vtjnash vtjnash commented Oct 3, 2025

Remove hash method definition when Base.memhash is not available.
On Julia 1.13+, these AbstractString types will use the default
AbstractString hash implementation which is now efficient and
zero-copy based on codeunit/iterate.

For Julia <1.13, continue using the memhash-based implementation
for compatibility.

Related to JuliaLang/julia#59697

🤖 Generated with Claude Code

Co-Authored-By: Claude noreply@anthropic.com

Remove hash method definition when Base.memhash is not available.
On Julia 1.13+, these AbstractString types will use the default
AbstractString hash implementation which is now efficient and
zero-copy based on codeunit/iterate.

For Julia <1.13, continue using the memhash-based implementation
for compatibility.

Related to JuliaLang/julia#59697

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude <noreply@anthropic.com>
@vtjnash vtjnash force-pushed the fix-memhash-compat branch from 62c4f4e to d9c68d1 Compare October 3, 2025 19:23
@vtjnash vtjnash closed this Feb 14, 2026
@vtjnash vtjnash reopened this Feb 14, 2026
@vtjnash
Copy link
Contributor Author

vtjnash commented Feb 14, 2026

bump? doesn't really matter to me if you make a release, just merging to get it out of my commit list would be nice (looks like you can add dependabot.yml too if you want CI healthy again)

$ less .github/dependabot.yml 

version: 2
updates:
  - package-ecosystem: "github-actions"
    directory: "/"
    schedule:
      interval: "monthly"
    open-pull-requests-limit: 99
    labels:
      - "dependencies"
      - "github-actions"

@tpapp
Copy link
Owner

tpapp commented Feb 14, 2026

Sorry, I overlooked this; I appreciate the fix and will make a release. Thanks for the reminder!

@tpapp tpapp merged commit 5f060d4 into tpapp:master Feb 14, 2026
0 of 6 checks passed
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