-
Notifications
You must be signed in to change notification settings - Fork 4.2k
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
.sql heuristic falls back to SQL #4888
Conversation
This pull request has been automatically marked as stale because it has not had recent activity, and will be closed if no further activity occurs. If this pull request was overlooked, forgotten, or should remain open for any other reason, please reply here to call attention to it and remove the stale status. Thank you for your contributions. |
This pull request has been automatically marked as stale because it has not had recent activity, and will be closed if no further activity occurs. If this pull request was overlooked, forgotten, or should remain open for any other reason, please reply here to call attention to it and remove the stale status. Thank you for your contributions. |
- add SQL as fallback for .sql extension. - SQL PL: add SYSCAT.|SYSPROC.|ASSOCIATE RESULT SET|END\! - T-SQL: [dbo]
* .sql falls back to SQL - add SQL as fallback for .sql extension. - SQL PL: add SYSCAT.|SYSPROC.|ASSOCIATE RESULT SET|END\! - T-SQL: [dbo] * improve sql heuristics
Description
This is part of a series of PRs for SQL, see #4884.
Do not use the classifier strategy for
.sql
, fallback to SQL. The premise here is that classifying any SQL variant as SQL is less wrong than classifying it as the wrong variant.Adds some new patterns to SQL PL and T-SQL to keep everything working, at least for current samples.
Checklist: