Skip to content

Fix Windows path syntax in documentation. Fix assorted typos in codebase.#5059

Merged
cobaltt7 merged 3 commits intopsf:mainfrom
hunterhogan:Windows_path
Mar 21, 2026
Merged

Fix Windows path syntax in documentation. Fix assorted typos in codebase.#5059
cobaltt7 merged 3 commits intopsf:mainfrom
hunterhogan:Windows_path

Conversation

@hunterhogan
Copy link
Contributor

  1. scripts/fuzz.py: fixed typo in FUNCTION NAME. I searched for references to the function, and the only references were within the module.

  2. What motivated starting this PR: in docs, fix Windows path syntax. C:\\ will usually work because the OS treats consecutive separators as only one separator. Nevertheless, C:\ is the correct syntax. See, e.g., https://learn.microsoft.com/en-us/windows/win32/fileio/naming-a-file

  3. Since I had the repo open, I fixed some typos. Other than the typo above, all fixes were in comments and docstrings.

hunterhogan and others added 3 commits March 20, 2026 16:49
1. scripts/fuzz.py: fixed typo in FUNCTION NAME. I searched for references to the function, and the only references were within the module.
2. What motivated starting this PR: in docs, fix Windows path syntax. `C:\\` will usually work because the OS treats consecutive separators as only one separator. Nevertheless, `C:\` is the correct syntax. _See, e.g.,_ https://learn.microsoft.com/en-us/windows/win32/fileio/naming-a-file
3. Since I had the repo open, I fixed some typos. Other than the typo above, all fixes were in comments and docstrings.
@cobaltt7 cobaltt7 added the ci: skip news Pull requests that don't need a changelog entry. label Mar 21, 2026
@github-actions
Copy link
Contributor

diff-shades results comparing this PR (e9f0ee7) to main (9dff4e4):

--preview style: no changes

--stable style: no changes


What is this? | Workflow run | diff-shades documentation

@cobaltt7 cobaltt7 merged commit 21bb2fc into psf:main Mar 21, 2026
67 of 68 checks passed
@hunterhogan hunterhogan deleted the Windows_path branch March 21, 2026 20:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci: skip news Pull requests that don't need a changelog entry.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants