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

[C#] Drop NAnt Build File syntax definition #3899

Merged
merged 1 commit into from
Dec 30, 2023

Conversation

FichteFoll
Copy link
Collaborator

It is basically a very rudimentary XML syntax that does not bring anything to the table, does not have tests and does not have any other integration within ST either. Because of the generic file extension, and 100% overlap with the first_line_match, it cannot even be reasonably used as a syntax definition by default and compared to the standard XML syntax definition, which would automatically be selected in 99% of all ST environments, it's just worse in every way.

See also #3569 for a previous discussion.

It is basically a very rudimentary XML syntax that does not bring
anything to the table, does not have tests and does not have any other
integration within ST either. Because of the generic file extension,
and 100% overlap with the `first_line_match`, it cannot even be
reasonably used as a syntax definition by default and compared to the
standard XML syntax definition, which would automatically be selected
in 99% of all ST environments, it's just worse in every way.

See also sublimehq#3569 for a previous discussion.
@FichteFoll FichteFoll merged commit 215bba1 into sublimehq:master Dec 30, 2023
2 checks passed
@FichteFoll FichteFoll deleted the pr/c-sharp/remove-nant branch December 30, 2023 13:52
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

Successfully merging this pull request may close these issues.

3 participants