Skip to content

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.

4 participants