fix spelling in tor.md docs #32961

pull stutxo wants to merge 1 commits into bitcoin:master from stutxo:spelling_mistake_tor changing 1 files +1 −1

stutxo commented at 10:17 PM on July 13, 2025: contributor

This PR is to fix some spelling mistakes i found of the word occurrences! there are two occurrences of this mistake.

thanks!
DrahtBot commented at 10:18 PM on July 13, 2025: contributor



The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.



Code Coverage & Benchmarks

For details see: https://corecheck.dev/bitcoin/bitcoin/pulls/32961.



Reviews

See the guideline for information on the review process.

Type Reviewers

ACK delta1, maflcko, willcl-ark

If your review is incorrectly listed, please react with 👎 to this comment and the bot will ignore it on the next update.

Type	Reviewers
ACK	delta1, maflcko, willcl-ark

in src/secp256k1/src/wycheproof/ecdsa_secp256k1_sha256_bitcoin_test.json:13 in b1522adfc8 outdated

   9 | @@ -10,7 +10,7 @@
  10 |    "notes" : {
  11 |      "ArithmeticError" : {
  12 |        "bugType" : "EDGE_CASE",
  13 | -      "description" : "Some implementations of ECDSA have arithmetic errors that occur when intermediate results have extreme values. This test vector has been constructed to test such occurences.",
  14 | +      "description" : "Some implementations of ECDSA have arithmetic errors that occur when intermediate results have extreme values. This test vector has been constructed to test such occurrences.",

stutxo commented at 10:21 PM on July 13, 2025:

it has come to my attention that i cant change this file, so i can remove this one if thats a problem

stutxo commented at 10:26 PM on July 13, 2025:

I removed the occurrence of the spelling mistake in this commit https://github.com/bitcoin/bitcoin/commit/84ef5524d5abaf07cc9970f42e74ee15bd381e3d

delta1 commented at 7:13 AM on July 14, 2025:

I think this change would have to be upstream at bitcoin-core/secp256k1

fix spelling in tor.md docs 84ef5524d5
stutxo force-pushed on Jul 13, 2025
stutxo renamed this:
~~fix spelling in tor.md docs and wycheproof description~~
fix spelling in tor.md docs
on Jul 13, 2025
DrahtBot added the label CI failed on Jul 13, 2025
DrahtBot commented at 10:26 PM on July 13, 2025: contributor


🚧 At least one of the CI tasks failed. <sub>Task lint: https://github.com/bitcoin/bitcoin/runs/45887606487</sub> <sub>LLM reason (✨ experimental): The CI failure is caused by errors in the lint step, specifically related to subtree merging issues.</sub>

<details><summary>Hints</summary>

Try to run the tests locally, according to the documentation. However, a CI failure may still happen due to a number of reasons, for example:
- Possibly due to a silent merge conflict (the changes in this pull request being incompatible with the current code in the target branch). If so, make sure to rebase on the latest commit of the target branch.
- A sanitizer issue, which can only be found by compiling with the sanitizer and running the affected test.
- An intermittent issue.
Leave a comment here, if you need help tracking down a confusing failure.

</details>
DrahtBot removed the label CI failed on Jul 13, 2025
delta1 commented at 4:33 AM on July 14, 2025: none

ACK 84ef5524d5abaf07cc9970f42e74ee15bd381e3d

just a typo fix in a markdown file
maflcko commented at 6:17 AM on July 14, 2025: member

lgtm ACK 84ef5524d5abaf07cc9970f42e74ee15bd381e3d
willcl-ark commented at 9:04 AM on July 14, 2025: member

ACK 84ef5524d5abaf07cc9970f42e74ee15bd381e3d
fanquake merged this on Jul 14, 2025
fanquake closed this on Jul 14, 2025
stringintech referenced this in commit f19e7819e9 on Jul 19, 2025
alexanderwiederin referenced this in commit 9997c3d993 on Jul 25, 2025
alexanderwiederin referenced this in commit 8d6ebb9c4e on Jul 28, 2025
alexanderwiederin referenced this in commit 533a4585b5 on Jul 28, 2025
stutxo deleted the branch on Aug 17, 2025

Contributors

fix spelling in tor.md docs #32961

Code Coverage & Benchmarks

Reviews