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

Newline issue #34

Closed
jasonkuhrt opened this issue Jun 21, 2021 · 2 comments
Closed

Newline issue #34

jasonkuhrt opened this issue Jun 21, 2021 · 2 comments
Labels
status: waiting for author Needs an action taken by the original poster

Comments

@jasonkuhrt
Copy link

Started here indentjs/endent#8

More details here graphql-nexus/nexus-prisma#50

The issue is that if \n shows up in a template string it gets read as a newline.

@JoshuaKGoldberg
Copy link
Collaborator

JoshuaKGoldberg commented May 22, 2023

There's a lot of text to read through in those two linked issues & their linked issues, and I lost track of where the problem is in this dedent specifically. Could you/someone please post a concrete example of using dedent in a way that shows the bug - including expected & actual outputs for some input?

This might be related to / even a duplicate of #22 -> #12.

@JoshuaKGoldberg JoshuaKGoldberg added the status: waiting for author Needs an action taken by the original poster label May 22, 2023
@JoshuaKGoldberg
Copy link
Collaborator

Character escaping logic was reworked a bit in #2 / #45 / #46 / #65. Since this one has been sitting for a bit and might be a dup of #12, closing out to keep the issue queue small. Please let me know if that's wrong! ❤️

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: waiting for author Needs an action taken by the original poster
Projects
None yet
Development

No branches or pull requests

2 participants