Skip to content

Add .sql as a plain text input format - #611

Merged
KnifeLemon merged 6 commits into
KnifeLemon:mainfrom
adnan-bhaldar:feat/sql-as-text-607
Oct 26, 2026
Merged

KnifeLemon merged 6 commits into
KnifeLemon:mainfrom
adnan-bhaldar:feat/sql-as-text-607

Conversation

@adnan-bhaldar

@adnan-bhaldar adnan-bhaldar commented Oct 26, 2026 •

Copy link
Copy Markdown
Contributor

What does this change?

Adds sql to the extensions of the TXT format, so .sql files (for example scripts saved from SSMS) are read as plain text, the same way .py, .bat, .lua and .ps1 already are. The files are only read, never executed.

Fixes #607

Type of change

  • Bug fix
  • New feature or improvement
  • New format or engine
  • Translation
  • Documentation
  • Build, CI or dependencies

How was it tested?

  • dotnet format --verify-no-changes
  • dotnet test
  • Tried it in the app (what you converted or clicked, and on which OS): converted .sql files saved as UTF-16 (BOM), UTF-8 (BOM) and Windows ANSI to TXT on Windows, and checked that accented characters (é, ü) are correct in the .txt

Screenshots (UI changes)

None, no UI changes.

Checklist

  • New UI strings are in all src/Filee.App/Assets/i18n/*.json files (no new UI strings)
  • New third-party code or binaries are listed in THIRD-PARTY-NOTICES.md (none added)
  • The PR does one thing (unrelated changes go in a separate PR)

SQL scripts (e.g. saved from SSMS) can now be converted to TXT, and from
there to PDF, DOCX and the other text routes. The extension is added to the
existing plain text format; no new converter is needed.
@adnan-bhaldar
adnan-bhaldar marked this pull request as draft October 26, 2026 07:25
…ifeLemon#607)

The extra converter had no description in the i18n files, so the Engines
page test failed. .sql is now handled by adding it to the existing txt
format, like .py and .bat.
@adnan-bhaldar
adnan-bhaldar marked this pull request as ready for review October 26, 2026 11:47
@KnifeLemon
KnifeLemon merged commit bb2cd3d into KnifeLemon:main Oct 26, 2026
8 checks passed
@KnifeLemon KnifeLemon mentioned this pull request Oct 26, 2026
1 task done
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.

[Format]: .sql

2 participants