-
Notifications
You must be signed in to change notification settings - Fork 91
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
Support xml.format.enforceQuoteStyle setting with experimental formatter #1249
Conversation
org.eclipse.lemminx/src/main/java/org/eclipse/lemminx/dom/DTDDeclNode.java
Outdated
Show resolved
Hide resolved
...eclipse.lemminx/src/main/java/org/eclipse/lemminx/services/format/DOMAttributeFormatter.java
Outdated
Show resolved
Hide resolved
org.eclipse.lemminx/src/test/java/org/eclipse/lemminx/XMLAssert.java
Outdated
Show resolved
Hide resolved
org.eclipse.lemminx/src/main/java/org/eclipse/lemminx/services/format/DOMDocTypeFormatter.java
Show resolved
Hide resolved
46a61a1
to
293e60e
Compare
...eclipse.lemminx/src/main/java/org/eclipse/lemminx/services/format/DOMAttributeFormatter.java
Outdated
Show resolved
Hide resolved
org.eclipse.lemminx/src/main/java/org/eclipse/lemminx/services/format/DOMDocTypeFormatter.java
Outdated
Show resolved
Hide resolved
org.eclipse.lemminx/src/main/java/org/eclipse/lemminx/services/format/DOMDocTypeFormatter.java
Show resolved
Hide resolved
org.eclipse.lemminx/src/main/java/org/eclipse/lemminx/services/format/DOMDocTypeFormatter.java
Outdated
Show resolved
Hide resolved
org.eclipse.lemminx/src/main/java/org/eclipse/lemminx/services/format/DOMDocTypeFormatter.java
Outdated
Show resolved
Hide resolved
org.eclipse.lemminx/src/main/java/org/eclipse/lemminx/services/format/DOMDocTypeFormatter.java
Show resolved
Hide resolved
org.eclipse.lemminx/src/main/java/org/eclipse/lemminx/services/format/DOMDocTypeFormatter.java
Show resolved
Hide resolved
org.eclipse.lemminx/src/main/java/org/eclipse/lemminx/services/format/DOMDocTypeFormatter.java
Show resolved
Hide resolved
org.eclipse.lemminx/src/main/java/org/eclipse/lemminx/services/format/DOMDocTypeFormatter.java
Show resolved
Hide resolved
org.eclipse.lemminx/src/main/java/org/eclipse/lemminx/services/format/DOMDocTypeFormatter.java
Show resolved
Hide resolved
org.eclipse.lemminx/src/main/java/org/eclipse/lemminx/services/format/TextEditUtils.java
Outdated
Show resolved
Hide resolved
...eclipse.lemminx/src/main/java/org/eclipse/lemminx/services/format/DOMAttributeFormatter.java
Show resolved
Hide resolved
...c/test/java/org/eclipse/lemminx/services/format/experimental/XMLFormatterQuoteStyleTest.java
Show resolved
Hide resolved
org.eclipse.lemminx/src/main/java/org/eclipse/lemminx/services/format/TextEditUtils.java
Outdated
Show resolved
Hide resolved
...eclipse.lemminx/src/main/java/org/eclipse/lemminx/services/format/DOMAttributeFormatter.java
Outdated
Show resolved
Hide resolved
...c/test/java/org/eclipse/lemminx/services/format/experimental/XMLFormatterQuoteStyleTest.java
Show resolved
Hide resolved
...eclipse.lemminx/src/main/java/org/eclipse/lemminx/services/format/DOMAttributeFormatter.java
Outdated
Show resolved
Hide resolved
I think that there is also a |
Ohh I completely missed that, thank you! |
org.eclipse.lemminx/src/main/java/org/eclipse/lemminx/services/format/DOMDocTypeFormatter.java
Outdated
Show resolved
Hide resolved
org.eclipse.lemminx/src/main/java/org/eclipse/lemminx/services/format/DOMDocTypeFormatter.java
Show resolved
Hide resolved
org.eclipse.lemminx/src/main/java/org/eclipse/lemminx/services/format/TextEditUtils.java
Show resolved
Hide resolved
org.eclipse.lemminx/src/test/java/org/eclipse/lemminx/XMLAssert.java
Outdated
Show resolved
Hide resolved
Once my comments will be fixed, the code looks good. @datho7561 could you please test the pr too. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This works very well for me. Well done, Jessica! Once you've addressed Angelo's comments, I'll merge it.
Signed-off-by: Jessica He <[email protected]>
Once this pr will be meŕged @JessicaJHee could yiu create a pr in vscode xml in Format.md and package.json to remove comment which says that your settings is now supported. |
Fixes #1239
Signed-off-by: Jessica He [email protected]