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

Add syntax highlight code for log files #3105

Merged
merged 1 commit into from
Mar 13, 2024

Conversation

taconi
Copy link
Contributor

@taconi taconi commented Dec 29, 2023

Adding log file support by default so you don't need to install the language-log plugin.

image

runtime/syntax/log.yaml Outdated Show resolved Hide resolved
runtime/syntax/log.yaml Outdated Show resolved Hide resolved
@JoeKar JoeKar merged commit 14dca7d into zyedidia:master Mar 13, 2024
3 checks passed
@taconi taconi deleted the syntax-highlighting/log branch March 13, 2024 23:55
- identifier: "^(\\d{4}[-/]?\\d{2}[-/]?\\d{2}\\s\\d{2}:\\d{2}(:\\d{2})?(\\.?\\d{2,8})?)"
- identifier: "^([0-2][0-9]|[0-2]-?[0-9][0-9]-?[0-9][0-9])\\-([0-1][0-9])\\-([0-3][0-9]) ([0-2][0-9])\\:([0-5][0-9])\\:([0-5][0-9]),([0-9][0-9][0-9])"
# ISO 8601:2004(E)
- identifier: ""
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

With #3253 this line came up after greping over the current syntax definitions.
From my current perspective this isn't needed or do I miss something right now?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It must have been some test I forgot to take, it wasn't supposed to be like this.

Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ok, then we solve this with #3256, to which you already respond.

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.

2 participants