Skip to content

🐛 Fix copier to handle string vars with spaces in quotes#631

Merged
tiangolo merged 7 commits intomasterfrom
FL-37-handle-string-with-spaces-copier
Mar 7, 2024
Merged

🐛 Fix copier to handle string vars with spaces in quotes#631
tiangolo merged 7 commits intomasterfrom
FL-37-handle-string-with-spaces-copier

Conversation

@estebanx64
Copy link
Copy Markdown
Contributor

No description provided.

@tiangolo
Copy link
Copy Markdown
Member

tiangolo commented Mar 7, 2024

Thanks! I updated it to inline the quotes using an f-string with f"{upper_key}={value!r}", the trick is the !r that makes it print the quotes. 🤓

@tiangolo tiangolo changed the title Fixed copier to handle string vars with spaces without quotes 🐛 Fix copier to handle string vars with spaces in quotes Mar 7, 2024
@tiangolo tiangolo enabled auto-merge (squash) March 7, 2024 17:50
@tiangolo tiangolo merged commit a44d2fd into master Mar 7, 2024
@tiangolo tiangolo deleted the FL-37-handle-string-with-spaces-copier branch March 7, 2024 17:51
gusevyaroslove pushed a commit to gusevyaroslove/fastapi-template that referenced this pull request Aug 4, 2024
Co-authored-by: Sebastián Ramírez <tiangolo@gmail.com>
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