Skip to content
This repository has been archived by the owner on Sep 20, 2023. It is now read-only.

Implemented basic Zig syntax checker. #2218

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

DavidYKay
Copy link

Implemented a very simple checker for the [Zig language].

Implementation currently invokes "Zig Build" and thus depends on using a Zig buildfile.

Does not yet support GetHighlightRegex.

Depends on using a Zig buildfile.
@lcd047
Copy link
Collaborator

lcd047 commented Oct 21, 2018

The Zig language seems to be experimental. Please consider turning your patch into an external checker. This is the intended use for external checkers: the syntastic checker should be distributed with some kind of package that adds support for Zig files to Vim. This way the checker can be maintained by people who actually use the language and care about it.

@DavidYKay
Copy link
Author

OK, I ended up switching to ALE for my own personal use. Happy to move this to an external checker if someone else pipes in and wants this merged into Syntastic. Otherwise, I'll most likely just leave this PR as-is.

Thanks much for your reply and your work on Syntastic! 😄

@lcd047
Copy link
Collaborator

lcd047 commented Oct 28, 2018

Absolutely, ALE is the way to go.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants