Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Merged
merged 2 commits into from
Nov 12, 2023

Conversation

madduck
Copy link
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
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
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
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