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

Create a stringifier for "quoted" values #1514

Merged
merged 1 commit into from
Dec 26, 2024

Conversation

henriquemoody
Copy link
Member

The StandardQuoter adds backticks around strings, which indicates that it's not a simple string but a code. With this stringifier, we can add quotes to placeholders directly into templates.

@henriquemoody henriquemoody added this to the 3.0 milestone Dec 26, 2024
The `StandardQuoter` adds backticks around strings, which indicates that
it's not a simple string but a code. With this stringifier, we can add
quotes to placeholders directly into templates.
Copy link

codecov bot commented Dec 26, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.60%. Comparing base (b8f4949) to head (4379f66).
Report is 1 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff              @@
##               main    #1514      +/-   ##
============================================
+ Coverage     96.59%   96.60%   +0.01%     
- Complexity      979      986       +7     
============================================
  Files           201      203       +2     
  Lines          2435     2448      +13     
============================================
+ Hits           2352     2365      +13     
  Misses           83       83              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@henriquemoody henriquemoody merged commit 4379f66 into Respect:main Dec 26, 2024
8 checks passed
@henriquemoody henriquemoody deleted the core/quoter branch January 17, 2025 09:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant