Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: use %q instead of manual \"%s\" #1723

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Commits on Oct 3, 2024

  1. fix: use %q instead of manual \"%s\"

    Noticed in a post-codereview of PR quicksilver-zone#1704 that there
    was a manual value quoting usinq `\"%s\"` instead
    of the format specifier `%q`. Manual quoting doesn't
    work correctly. One can find vestiges of it by
    
        git grep '\\"%s'
    
    Updates quicksilver-zone#1704
    odeke-em committed Oct 3, 2024
    Configuration menu
    Copy the full SHA
    282d564 View commit details
    Browse the repository at this point in the history

Commits on Oct 9, 2024

  1. Configuration menu
    Copy the full SHA
    e5ecaa6 View commit details
    Browse the repository at this point in the history