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

Linting for embedded Javascript #262

Closed
LogMull opened this issue Feb 17, 2023 · 2 comments
Closed

Linting for embedded Javascript #262

LogMull opened this issue Feb 17, 2023 · 2 comments
Labels
enhancement New feature or request

Comments

@LogMull
Copy link

LogMull commented Feb 17, 2023

Is there an existing way to run any kind of linting against embedded javascript in objectscript classes? If not, is that something that is planned / reasonable to expect in the future? My team makes heavy use of the embedded javascript and not having any kind of linting on it has been a pain point.

@isc-bsaviano isc-bsaviano added the enhancement New feature or request label Feb 17, 2023
@isc-bsaviano
Copy link
Contributor

@LogMull This isn't something that is likely to be implemented anytime soon. We added support for some intellisense in version 2.1.0 but linting would be much more difficult because most embedded JS is "infected" with embedded ObjectScript and any existing JS linter wouldn't handle that well. For things like hover and code completion that's not a big deal since you just won't get intellisense in some places, but if you get a huge number of false problems reported that is a very bad user experience.

@LogMull
Copy link
Author

LogMull commented Feb 17, 2023

I assumed it was not on the radar due to reasons like that. Thanks for the clarification and quick response!

@isc-bsaviano isc-bsaviano closed this as not planned Won't fix, can't repro, duplicate, stale Feb 18, 2023
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

No branches or pull requests

2 participants