Fix more whirlpool typos (#47)

[dfandrich]

Summary by CodeRabbit

• Bug Fixes
  • Corrected the spelling of the Whirlpool hash algorithm ID from "whirpool" to "whirlpool" in documentation and algorithm definitions.

[coderabbitai]

📝 Walkthrough

Walkthrough

The pull request corrects a typographical error in the Whirlpool hash function definition. The algorithm ID field is updated from the misspelled "whirpool" to the correct spelling "whirlpool" across two files: the README documentation table and the YAML configuration file.

Changes

Cohort / File(s)	Summary
Documentation Update definitions_crypto_algorithms/README.md	Corrected algorithm ID typo in Hash Function table from "whirpool" to "whirlpool".
Configuration File Update definitions_crypto_algorithms/algorithms/Hash Functions/whirlpool.yaml	Corrected the algorithmId field value from "whirpool" to "whirlpool".

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

• Update and rename whirpool.yaml to whirlpool.yaml #46: Fixes the identical typographical bug in algorithm ID for Whirlpool, with additional changes including file renaming and keyword updates.

Suggested reviewers

• matiasdaloia

Poem

🐰 A whisker-twitch of joy, I say,
For typos fixed without delay,
"Whirpool" → "Whirlpool" spins so right,
The hash algorithms dance in light! ✨

🚥 Pre-merge checks | ✅ 3

✅ Passed checks (3 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The pull request title accurately describes the main change: fixing typographical errors in 'whirlpool' spelled as 'whirpool' across the codebase.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[coderabbitai]

Actionable comments posted: 1

🤖 Fix all issues with AI agents

In `@definitions_crypto_algorithms/algorithms/Hash` Functions/whirpool.yaml:
- Around line 1-5: The file name contains a typo: rename the YAML file from
"whirpool.yaml" to "whirlpool.yaml" so it matches the algorithmId "whirlpool"
and avoids tooling/lookup issues; update any references to this file if present
(e.g., imports or index entries) to use "whirlpool.yaml" and confirm the
algorithmId and contents remain unchanged.

[coderabbitai]

⚠️ Potential issue | 🟡 Minor

🧩 Analysis chain

🏁 Script executed:

#!/bin/bash
# Search for any remaining "whirpool" references (paths, ids, docs)
rg -n "whirpool"

# Check for "whirlpool" references to ensure consistency
rg -n "whirlpool"

Repository: scanoss/crypto_algorithms_open_dataset

Length of output: 2225

---

Rename file to fix typo: whirpool.yaml → whirlpool.yaml.

The filename whirpool.yaml in the algorithms directory is a misspelling of "Whirlpool". The algorithmId is correctly set to whirlpool, but the filename should match. Rename the file to whirlpool.yaml to maintain consistency with the algorithmId and prevent tooling lookup issues.

🤖 Prompt for AI Agents

In `@definitions_crypto_algorithms/algorithms/Hash` Functions/whirpool.yaml around
lines 1 - 5, The file name contains a typo: rename the YAML file from
"whirpool.yaml" to "whirlpool.yaml" so it matches the algorithmId "whirlpool"
and avoids tooling/lookup issues; update any references to this file if present
(e.g., imports or index entries) to use "whirlpool.yaml" and confirm the
algorithmId and contents remain unchanged.