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

Consider adding linters #12

Open
AlekSi opened this issue Aug 9, 2017 · 1 comment
Open

Consider adding linters #12

AlekSi opened this issue Aug 9, 2017 · 1 comment

Comments

@AlekSi
Copy link

AlekSi commented Aug 9, 2017

Specifically, I would like to see go vet.

@iafan
Copy link
Owner

iafan commented Aug 10, 2017

Thanks for the suggestion, Alexey. Unfortunately, go vet is a command, not a package one can use directly from external code, so it makes it currently impossible to plug into Go Play Space (GPS), since GPS runs formatting/validation in the browser, without any backend code. The actual code execution is proxied to the official Go Playground, so if Go Playground adds an ability for linting, I'll happily support it. Note that some linters also require the file to be saved to disk in the proper directory relative to GOPATH, which might be also a potential obstacle in implementing linting for playground scripts.

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

No branches or pull requests

2 participants