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

alejandra modifies the logical contents of multiline strings #409

Open
srhb opened this issue Feb 20, 2024 · 0 comments
Open

alejandra modifies the logical contents of multiline strings #409

srhb opened this issue Feb 20, 2024 · 0 comments
Labels
component | engine Core algorithm of the tool roadmap | comment pragmas status | it is a good thing We agree it is good to implement this type | feature request New feature or request

Comments

@srhb
Copy link

srhb commented Feb 20, 2024

given the file:

# trailing-space-in-multiline-string.nix
''
  foo 
''

(note the trailing space character, foo )

alejandra reformats it:

# trailing-space-in-multiline-string.nix
''
  foo
''

... that is, it deletes the trailing space.

alejandra cannot know that this is safe for a given multiline string and shouldn't modify the actual contents.

@kamadorueda kamadorueda added type | feature request New feature or request status | it is a good thing We agree it is good to implement this component | engine Core algorithm of the tool roadmap | comment pragmas labels Nov 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component | engine Core algorithm of the tool roadmap | comment pragmas status | it is a good thing We agree it is good to implement this type | feature request New feature or request
Projects
None yet
Development

No branches or pull requests

2 participants