Skip to content

feat(crypto): restore ripemd160, sha1 and sha256 hash helpers #2248

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

landabaso
Copy link
Member

Why

Several downstream projects still rely on the simple hash helpers removed during the v7 refactor. Bringing them back avoids extra boilerplate and keeps parity with previous majors while we're still in RC.

What

  • Re‑added ripemd160, sha1 and sha256 helpers to crypto .
  • Adjusted tests & fixtures (test/crypto.spec.ts, test/fixtures/crypto.json).

Breaking changes?

None. Existing code keeps working.

@landabaso
Copy link
Member Author

landabaso commented Jul 24, 2025

CI is red due to a GitHub Actions cache hiccup:

Error: Cache service responded with 400
Warning: The `save-state` command is deprecated…

All unit + integration tests pass locally. The same 400 cache issue showed up in #2247. I’m digging into it...

@junderw
Copy link
Member

junderw commented Jul 26, 2025

Good work.

Re: CI changes, pin the commits to the v4 branch so we don't have to spend mental capacity weighing the options of changing pinning/no-pinning.

Default is that maintainers can modify PR branches, so you can go to their fork and edit that branch directly with a commit via the web UI if you want.

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