Sanitize more quotation marks, option to export lowercase#102
Merged
Spenhouet merged 2 commits intoSpenhouet:mainfrom Apr 7, 2026
Merged
Sanitize more quotation marks, option to export lowercase#102Spenhouet merged 2 commits intoSpenhouet:mainfrom
Spenhouet merged 2 commits intoSpenhouet:mainfrom
Conversation
Add more quotation marks to default configuration for sanitizer: "'", "’", "´", "`" Add new option to export all files/paths as lowercase.
Contributor
Author
|
Unfortunately RUFF didn't like some of the new quotation marks added, so I had to exclude that check for that line. More information about the warning: |
Spenhouet
approved these changes
Apr 7, 2026
Spenhouet
approved these changes
Apr 7, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add more quotation marks to default configuration for sanitizer: "'", "’", "´", "`"
Add new option to export all files/paths as lowercase.
Summary
When exporting a confluence area, I noticed that several documents where created included quotation marks.
These can cause problems when further post processing the exported files, like running them through a shell script.
In order to avoid these problems.
Also added a new option to make the filenames lowercase. When working with files some parties prefers working with files in lowercase only. (On Linux the casing matters, on Windows it doesn't)
Test Plan
Tested running export on internal confluence.
Added additional unit test to cover the downcasing case.