This is an automated email from the ASF dual-hosted git repository. beto pushed a commit to branch fix-sqlglot-linter in repository https://gitbox.apache.org/repos/asf/superset.git
commit 47e5772648b9a751d4e9fe8b9f0e7b2153e652ef Author: Beto Dealmeida <[email protected]> AuthorDate: Fri Jun 20 12:43:38 2025 -0700 fix: sqlglot linter --- superset/extensions/pylint.py | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/superset/extensions/pylint.py b/superset/extensions/pylint.py index deb0c86572..fc177884da 100644 --- a/superset/extensions/pylint.py +++ b/superset/extensions/pylint.py @@ -87,9 +87,12 @@ class SQLParsingLibraryImportChecker(BaseChecker): def _is_disallowed(self, file_path: Path, root_mod: str) -> bool: # True if sqlglot is imported outside superset/sql, # or if any forbidden library is imported anywhere - in_superset_sql = file_path.match("**/superset/sql/**/*.py") or file_path.match( - "**/superset/sql/*.py" - ) + allowed = { + "**/supersql/sql/**/*.py", + "**/supersql/sql/*.py", + "**/superset/config.py", + } + in_superset_sql = any(file_path.match(pattern) for pattern in allowed) return (root_mod == "sqlglot" and not in_superset_sql) or root_mod in { "sqlparse", "sqloxide",
