BIP38, help GitHub intermediate syntax highlighter #1586

pull katesalazar wants to merge 2 commits into bitcoin:master from katesalazar:20240504 changing 1 files +4 −4
  1. katesalazar commented at 3:25 pm on May 4, 2024: contributor

    It seems GitHub has some problem interpreting quintuple single quotes on the blame and styles-enriched-raw renderings.

    Edited using the browser, pending rebase and fixup.

  2. katesalazar commented at 3:30 pm on May 4, 2024: contributor
    I’d prefer reporting upstream before here, but IDK how.
  3. murchandamus commented at 2:57 pm on May 6, 2024: contributor
    I understand the first change where you add the missing double single quotes to close the italicized phrase, but it’s not clear to me what problem this generally solves, since the before and after render the same for me. Could you clarify your motivation for this PR?
  4. katesalazar commented at 7:12 pm on May 6, 2024: contributor

    Yeah, I actually thought about adding some screenshots. :sweat_smile:

    First try to find something wrong here:

    2024-05-06_19-09

    If you were able to find something wrong in the previous figure, then try to find a fix here:

    2024-05-06_19-10

  5. murchandamus commented at 8:24 pm on May 6, 2024: contributor
    I see. I didn’t notice that the Specification section was italicized before.
  6. murchandamus commented at 1:38 pm on May 7, 2024: contributor
    Okay, please feel free to ping me for review when this is ready to be merged.
  7. Update bip-0038.mediawiki
    Add missing closing double single quote. The italicized paragraph gets
    cautiously closed.
    4ddb0cc893
  8. Update bip-0038.mediawiki
    Separating the bold and the italic markup helps inconsistent parsing
    (see screenshots in PR #1586).
    c88a018409
  9. katesalazar force-pushed on May 7, 2024
  10. katesalazar marked this as ready for review on May 7, 2024
  11. katesalazar commented at 3:58 pm on May 7, 2024: contributor
    Waiting for further feedback, I see no urgency justifying pinging.
  12. murchandamus commented at 12:49 pm on May 8, 2024: contributor
    This fixes a minor formatting issue that appears in the GitHub online editor with quintuple quotes and closes an italicized phrase by adding the missing closing symbol.
  13. murchandamus merged this on May 8, 2024
  14. murchandamus closed this on May 8, 2024


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: 2024-12-30 17:10 UTC

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