Skip to content

remove io/Filesystem - #9260

Merged
nbolton merged 4 commits into
masterfrom
fsUnused
Dec 8, 2025
Merged

remove io/Filesystem#9260
nbolton merged 4 commits into
masterfrom
fsUnused

Conversation

@sithlord48

@sithlord48 sithlord48 commented Dec 5, 2025

Copy link
Copy Markdown
Member

remove io/Filesystem

  • move the single use of fopenUtf8Path into the place its used.
    • secureUtils needs to use a FILE to write our cert. We dont have Qt methods for this but we want to use QFile everywhere else we touch a file.

Tested by generating a certificate on,

  • Linux
  • Windows

@sithlord48 sithlord48 changed the title chore: io/FileSystem remove unused methods remove io/Filesystem Dec 5, 2025
@sithlord48
sithlord48 marked this pull request as ready for review December 5, 2025 21:48
@sithlord48
sithlord48 requested a review from a team December 5, 2025 21:48
@sithlord48
sithlord48 requested a review from nbolton as a code owner December 5, 2025 21:48
@sithlord48
sithlord48 force-pushed the fsUnused branch 3 times, most recently from 851a753 to 9dae91c Compare December 5, 2025 22:00
@nbolton
nbolton merged commit 13750f4 into master Dec 8, 2025
34 checks passed
@nbolton
nbolton deleted the fsUnused branch December 8, 2025 11:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants