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.

Sign up for GitHub

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

Jump to bottom

fix: add types to DatasetReference constructor #1601

Merged
merged 13 commits into from
Apr 12, 2024
Merged

fix: add types to DatasetReference constructor #1601

Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
ebf6952
fix: add types to DatasetReference constructor
karelserruys-foodpairing Jun 29, 2023
06f689b
Merge branch 'main' into fix/datasetreference-constructor-types
chalmerlowe Jun 29, 2023
ed2097f
fix: add types to DatasetReference constructor
karelserruys-foodpairing Jul 6, 2023
b20a5c1
Merge branch 'fix/datasetreference-constructor-types' of https://gith…
karelserruys-foodpairing Jul 6, 2023
0090f77
Merge branch 'main' into fix/datasetreference-constructor-types
chalmerlowe Jul 14, 2023
008a317
fix: DatasetReference.from_string test coverage
karelserruys-foodpairing Aug 21, 2023
38f2fbc
Merge branch 'main' into fix/datasetreference-constructor-types
kserruys Aug 21, 2023
e07cc59
Merge branch 'main' into fix/datasetreference-constructor-types
meredithslota Sep 1, 2023
bc71921
Merge branch 'main' into fix/datasetreference-constructor-types
meredithslota Sep 22, 2023
347427d
Merge branch 'main' into fix/datasetreference-constructor-types
meredithslota Nov 8, 2023
269dcd6
Merge branch 'main' into fix/datasetreference-constructor-types
tswast Apr 11, 2024
abfcc94
Merge branch 'main' into fix/datasetreference-constructor-types
tswast Apr 12, 2024
e23cf4b
Merge branch 'main' into fix/datasetreference-constructor-types
tswast Apr 12, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Jump to
Jump to file
Failed to load files.
Diff view
Diff view
Prev Previous commit
Next Next commit
fix: add types to DatasetReference constructor
  • Loading branch information
@karelserruys-foodpairing
karelserruys-foodpairing committed Jul 6, 2023
commit ed2097fc80bb6a1de1e8385d53e3b73e81b471fa
20 changes: 10 additions & 10 deletions google/cloud/bigquery/dataset.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,7 @@ class DatasetReference(object):
ValueError: If either argument is not of type ``str``.
"""

def __init__(self, project, dataset_id):
def __init__(self, project: str, dataset_id: str):
if not isinstance(project, str):
raise ValueError("Pass a string for project")
if not isinstance(dataset_id, str):
Expand Down Expand Up @@ -166,15 +166,17 @@ def from_string(
standard SQL format.
"""
output_dataset_id = dataset_id
output_project_id = default_project
parts = _helpers._split_id(dataset_id)

if len(parts) == 1 and not default_project:
raise ValueError(
"When default_project is not set, dataset_id must be a "
"fully-qualified dataset ID in standard SQL format, "
'e.g., "project.dataset_id" got {}'.format(dataset_id)
)
if len(parts) == 1:
if default_project is not None:
output_project_id = default_project
else:
raise ValueError(
"When default_project is not set, dataset_id must be a "
"fully-qualified dataset ID in standard SQL format, "
'e.g., "project.dataset_id" got {}'.format(dataset_id)
)
elif len(parts) == 2:
output_project_id, output_dataset_id = parts
elif len(parts) > 2:
Expand All @@ -183,7 +185,6 @@ def from_string(
"dataset ID in standard SQL format. e.g. "
'"project.dataset_id", got {}'.format(dataset_id)
)

return cls(output_project_id, output_dataset_id)

def to_api_repr(self) -> dict:
Expand Down Expand Up @@ -454,7 +455,6 @@ def __ne__(self, other):
return not self == other

def __repr__(self):

return f"<AccessEntry: role={self.role}, {self._entity_type}={self.entity_id}>"

def _key(self):
Expand Down