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

Syntax checker #293

Open
worksofliam opened this issue Oct 30, 2024 · 1 comment · May be fixed by #307
Open

Syntax checker #293

worksofliam opened this issue Oct 30, 2024 · 1 comment · May be fixed by #307
Labels
enhancement New feature or request
Milestone

Comments

@worksofliam
Copy link
Contributor

Initial work done here: https://gist.github.com/worksofliam/5aa61347b97dd5d33325f059244b8160

@worksofliam worksofliam added the enhancement New feature or request label Oct 30, 2024
@worksofliam worksofliam added this to the 2.0.0 milestone Oct 30, 2024
@worksofliam
Copy link
Contributor Author

From a colleague:

Without knowing some of the options the user has defined, you are going to have problems. With a hard coded naming mode of *SQL, anyone using / for qualification will be very frustrated. The name qualification rules are slightly different between *SQL and *SYS, so even when using the more forgiving . as a separator, a user will encounter confusing, and incorrect, errors.
The decimal point character and the string delimiter values are important parameters. By not supporting them, your solution gets a little less accurate, which will lead to false errors.

@worksofliam worksofliam linked a pull request Dec 19, 2024 that will close this issue
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.

1 participant