-
Notifications
You must be signed in to change notification settings - Fork 99
Initial prototype of an ESLint-based validator of the Elasticsearch specification #4400
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
base: main
Are you sure you want to change the base?
Conversation
Looks very promising @JoshMock ! Very good work. I will try to write some rules next week. It's probably worth experimenting with that foundation a bit in order to identify recurring patterns for whose it might be worth providing high level helper functions / abstractions for as part of the spec validator "SDK". |
I agree, and would be happy to add that, but I think it would require some discussion to make sure I (or whoever implements it) am correctly targeting the correct problems. It's probably a lot easier to for someone on the static language side to list those problems, because many of them we don't run into as often on the dynamic side. 😄 @flobernd Would it be worth merging this as-is so we can iterate on it, or do you think more SDK tooling is a hard requirement? |
@JoshMock Oh let's absolutely merge this first, before we work on the follow up tasks 🙂 That comment was not meant to get addressed immediately. |
I just approved! |
As titled. 🚀