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

Escape sequence inconsistencies, extension settings? #36

Open
06000208 opened this issue Apr 26, 2023 · 0 comments
Open

Escape sequence inconsistencies, extension settings? #36

06000208 opened this issue Apr 26, 2023 · 0 comments

Comments

@06000208
Copy link

06000208 commented Apr 26, 2023

The dotenv library on npm, a popular node.js implementation, doesn't require escaping quotes inside quotes or support the use of \t, per motdotla/dotenv#711. It does support \n, though.

This results in inconsistency between the extension and resulting parsed strings, with newlines being correct, while tabs, escaped backslashes, and quotes inside quotes aren't:

2023-04-26-Code-1682494117

On the issue linked above, they describe 3 other libs behavior, and they're also all inconsistent in some way with this extension.

So would extension settings to customize which escape sequences are counted be possible, or adequately deal with this? Is there a specification somewhere that these libs should be following?

(I do believe the extension's default styling is correct, for what it's worth, and that dotenv should support these things. I'm also aware that this extension is for a different lib.)

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

No branches or pull requests

1 participant