Skip to content

Replace sql"" with fr"" to ensure trailing spaces#2368

Merged
eikek merged 2 commits intoeikek:masterfrom
madduck:fix-ph-fts-update
Nov 12, 2023
Merged

Replace sql"" with fr"" to ensure trailing spaces#2368
eikek merged 2 commits intoeikek:masterfrom
madduck:fix-ph-fts-update

Conversation

@madduck
Copy link
Copy Markdown
Contributor

@madduck madduck commented Nov 10, 2023

Thanks to @eikek for the suggestion. This PR replaces all occurrences of sql"" with fr"" (cf.
interpolators.

Fixes: #2367

Thanks to @eikek for the suggestion. This PR replaces all occurrences of
`sql""` with `fr""` (cf.
[interpolators](https://tpolecat.github.io/doobie/docs/08-Fragments.html#whitespace-handling).

Fixes: eikek#2367
Signed-off-by: martin f. krafft <madduck@madduck.net>
@madduck
Copy link
Copy Markdown
Contributor Author

madduck commented Nov 10, 2023

FTR, I just tested this and can confirm that it now runs without an error.

@eikek
Copy link
Copy Markdown
Owner

eikek commented Nov 11, 2023

Hi, could you run the sbt command fix (or sbt fix from a shell) to fix the source formatting? thanks!!! 🙏🏼

Signed-off-by: martin f. krafft <madduck@madduck.net>
@madduck
Copy link
Copy Markdown
Contributor Author

madduck commented Nov 11, 2023

Done, @eikek

@eikek eikek merged commit c0ce1fa into eikek:master Nov 12, 2023
@eikek eikek added this to the Docspell 0.41.0 milestone Nov 12, 2023
@madduck madduck deleted the fix-ph-fts-update branch January 16, 2024 16:17
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.

Generated SQL is missing a space

2 participants