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 a LookML validator #413

Closed
joshtemple opened this issue Sep 14, 2021 · 1 comment · Fixed by #422
Closed

Add a LookML validator #413

joshtemple opened this issue Sep 14, 2021 · 1 comment · Fixed by #422
Assignees
Labels
Enhancement New feature or request

Comments

@joshtemple
Copy link
Collaborator

In some cases, a Looker user may commit LookML and request Spectacles validation for LookML that was not written in Looker's web IDE. In these cases, the ability to run LookML validation via Spectacles would be helpful because it would catch syntax errors that can cause confusing errors in other validators (like the SQL validator).

We should add a new validator, like spectacles syntax, or similar, that runs Looker's LookML validator via the API and returns the results.

@joshtemple joshtemple added the Enhancement New feature or request label Sep 14, 2021
@TimStrietzel
Copy link

Dropping in: I can confirm, that this problem can arise in real-world-scenarios where a user pull the code locally to perform e.g. many file / folder structure based operations and/or 'heavy copy-pasting' between different files.

In cases like this, a user might prefer the power of a good local text editor over the brower based Looker web IDE UI.

Looking forward to get the fix to this live in Spectacles :-)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants