chore: update string formatting to f-strings #1818

pull famouswizard wants to merge 1 commits into bitcoin:master from famouswizard:patch-2 changing 1 files +2 −2
  1. famouswizard commented at 4:39 pm on April 11, 2025: none

    updated the code to replace the old-style string formatting with f-strings for better readability and consistency. For example, this:

    0raise ValueError('aux_rand must be 32 bytes instead of %i.' % len(aux_rand))
    

    has been changed to:

    0raise ValueError(f'aux_rand must be 32 bytes instead of {len(aux_rand)}.')
    

    Similarly, all instances of old-style formatting, such as:

    0print(' * Signing test raised exception:', e)
    

    have been updated to:

    0print(f' * Signing test raised exception: {e}')
    
  2. chore: update string formatting to f-strings 0300bf6eab
  3. jonatack commented at 4:47 pm on April 11, 2025: member
    Thank you for your contribution. Normally, this patch would require review and sign-off by the BIP author. I’m not convinced the change is vital or worth the time asked from reviewers, and it also adds notification noise for people who follow this repository. Closing for now but happy to re-open if the BIP author or longstanding reviewers to this project are in favor of the change.
  4. jonatack closed this on Apr 11, 2025

  5. famouswizard deleted the branch on Apr 11, 2025

github-metadata-mirror

This is a metadata mirror of the GitHub repository bitcoin/bips. This site is not affiliated with GitHub. Content is generated from a GitHub metadata backup.
generated: 2025-04-19 01:10 UTC

This site is hosted by @0xB10C
More mirrored repositories can be found on mirror.b10c.me