Skip to content

Commit

Permalink
Removing the data source name lower case requirement
Browse files Browse the repository at this point in the history
  • Loading branch information
tombaeyens committed Sep 11, 2024
1 parent a6f85fe commit 82c2564
Showing 1 changed file with 0 additions and 4 deletions.
4 changes: 0 additions & 4 deletions soda/contracts/soda/contracts/impl/contract_data_source.py
Original file line number Diff line number Diff line change
Expand Up @@ -90,10 +90,6 @@ def __init__(self, data_source_yaml_file: YamlFile):
yaml_helper: yaml_helper = YamlHelper(yaml_file=data_source_yaml_file, logs=self.logs)
self.type = yaml_helper.read_string(self.data_source_yaml_dict, self._KEY_TYPE)
self.name = yaml_helper.read_string(self.data_source_yaml_dict, self._KEY_NAME)
if isinstance(self.name, str) and not re.match("[_a-z0-9]+", self.name):
self.logs.error(
f"Data source name must contain only lower case letters, numbers and underscores. Was {self.name}"
)

self.sql_dialect: SqlDialect = self._create_sql_dialect()

Expand Down

0 comments on commit 82c2564

Please sign in to comment.