Skip to content

Allow None for optional parameters #82

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

Merged
henribru merged 1 commit into from
Jul 5, 2026
Merged

Allow None for optional parameters #82

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
The table of contents is too big for display.
Diff view
Diff view
  •  
  •  
  •  
5 changes: 4 additions & 1 deletion create_stubs.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -205,12 +205,15 @@ def write_resource(writer, resource_name, resource, api, top_level=True) -> Set[
parameter_name = key2param(parameter_name)
if keyword.iskeyword(parameter_name):
continue
if parameter.get("required"):
required = parameter.get("required", False)
if required:
required_parameters.append(parameter_name)
type = get_type(parameter)
if parameter.get("repeated"):
assert "_list[" not in type and type != "_list"
type = f"{type} | _list[{type}]"
if not required:
type = f"{type} | None"
parameters[parameter_name] = type
ordered_parameters = [
key2param(param) for param in method.get("parameterOrder", [])
Expand Down
24 changes: 12 additions & 12 deletions googleapiclient-stubs/_apis/accessapproval/v1/resources.pyi
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -45,9 +45,9 @@ class AccessApprovalResource(googleapiclient.discovery.Resource):
self,
*,
parent: str,
filter: str = ...,
pageSize: int = ...,
pageToken: str = ...,
filter: str | None = ...,
pageSize: int | None = ...,
pageToken: str | None = ...,
**kwargs: typing.Any,
) -> ListApprovalRequestsResponseHttpRequest: ...
def list_next(
Expand All @@ -70,7 +70,7 @@ class AccessApprovalResource(googleapiclient.discovery.Resource):
*,
name: str,
body: AccessApprovalSettings = ...,
updateMask: str = ...,
updateMask: str | None = ...,
**kwargs: typing.Any,
) -> AccessApprovalSettingsHttpRequest: ...
def approvalRequests(self) -> ApprovalRequestsResource: ...
Expand Down Expand Up @@ -107,9 +107,9 @@ class AccessApprovalResource(googleapiclient.discovery.Resource):
self,
*,
parent: str,
filter: str = ...,
pageSize: int = ...,
pageToken: str = ...,
filter: str | None = ...,
pageSize: int | None = ...,
pageToken: str | None = ...,
**kwargs: typing.Any,
) -> ListApprovalRequestsResponseHttpRequest: ...
def list_next(
Expand All @@ -132,7 +132,7 @@ class AccessApprovalResource(googleapiclient.discovery.Resource):
*,
name: str,
body: AccessApprovalSettings = ...,
updateMask: str = ...,
updateMask: str | None = ...,
**kwargs: typing.Any,
) -> AccessApprovalSettingsHttpRequest: ...
def approvalRequests(self) -> ApprovalRequestsResource: ...
Expand Down Expand Up @@ -169,9 +169,9 @@ class AccessApprovalResource(googleapiclient.discovery.Resource):
self,
*,
parent: str,
filter: str = ...,
pageSize: int = ...,
pageToken: str = ...,
filter: str | None = ...,
pageSize: int | None = ...,
pageToken: str | None = ...,
**kwargs: typing.Any,
) -> ListApprovalRequestsResponseHttpRequest: ...
def list_next(
Expand All @@ -194,7 +194,7 @@ class AccessApprovalResource(googleapiclient.discovery.Resource):
*,
name: str,
body: AccessApprovalSettings = ...,
updateMask: str = ...,
updateMask: str | None = ...,
**kwargs: typing.Any,
) -> AccessApprovalSettingsHttpRequest: ...
def approvalRequests(self) -> ApprovalRequestsResource: ...
Expand Down
60 changes: 35 additions & 25 deletions googleapiclient-stubs/_apis/accesscontextmanager/v1/resources.pyi
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,8 @@ class AccessContextManagerResource(googleapiclient.discovery.Resource):
name: str,
accessLevelFormat: typing_extensions.Literal[
"LEVEL_FORMAT_UNSPECIFIED", "AS_DEFINED", "CEL"
] = ...,
]
| None = ...,
**kwargs: typing.Any,
) -> AccessLevelHttpRequest: ...
def list(
Expand All @@ -38,9 +39,10 @@ class AccessContextManagerResource(googleapiclient.discovery.Resource):
parent: str,
accessLevelFormat: typing_extensions.Literal[
"LEVEL_FORMAT_UNSPECIFIED", "AS_DEFINED", "CEL"
] = ...,
pageSize: int = ...,
pageToken: str = ...,
]
| None = ...,
pageSize: int | None = ...,
pageToken: str | None = ...,
**kwargs: typing.Any,
) -> ListAccessLevelsResponseHttpRequest: ...
def list_next(
Expand All @@ -53,7 +55,7 @@ class AccessContextManagerResource(googleapiclient.discovery.Resource):
*,
name: str,
body: AccessLevel = ...,
updateMask: str = ...,
updateMask: str | None = ...,
**kwargs: typing.Any,
) -> OperationHttpRequest: ...
def replaceAll(
Expand Down Expand Up @@ -90,8 +92,8 @@ class AccessContextManagerResource(googleapiclient.discovery.Resource):
self,
*,
parent: str,
pageSize: int = ...,
pageToken: str = ...,
pageSize: int | None = ...,
pageToken: str | None = ...,
**kwargs: typing.Any,
) -> ListAuthorizedOrgsDescsResponseHttpRequest: ...
def list_next(
Expand All @@ -104,7 +106,7 @@ class AccessContextManagerResource(googleapiclient.discovery.Resource):
*,
name: str,
body: AuthorizedOrgsDesc = ...,
updateMask: str = ...,
updateMask: str | None = ...,
**kwargs: typing.Any,
) -> OperationHttpRequest: ...

Expand All @@ -130,8 +132,8 @@ class AccessContextManagerResource(googleapiclient.discovery.Resource):
self,
*,
parent: str,
pageSize: int = ...,
pageToken: str = ...,
pageSize: int | None = ...,
pageToken: str | None = ...,
**kwargs: typing.Any,
) -> ListServicePerimetersResponseHttpRequest: ...
def list_next(
Expand All @@ -144,7 +146,7 @@ class AccessContextManagerResource(googleapiclient.discovery.Resource):
*,
name: str,
body: ServicePerimeter = ...,
updateMask: str = ...,
updateMask: str | None = ...,
**kwargs: typing.Any,
) -> OperationHttpRequest: ...
def replaceAll(
Expand Down Expand Up @@ -181,9 +183,9 @@ class AccessContextManagerResource(googleapiclient.discovery.Resource):
def list(
self,
*,
pageSize: int = ...,
pageToken: str = ...,
parent: str = ...,
pageSize: int | None = ...,
pageToken: str | None = ...,
parent: str | None = ...,
**kwargs: typing.Any,
) -> ListAccessPoliciesResponseHttpRequest: ...
def list_next(
Expand All @@ -196,7 +198,7 @@ class AccessContextManagerResource(googleapiclient.discovery.Resource):
*,
name: str,
body: AccessPolicy = ...,
updateMask: str = ...,
updateMask: str | None = ...,
**kwargs: typing.Any,
) -> OperationHttpRequest: ...
def setIamPolicy(
Expand Down Expand Up @@ -228,10 +230,10 @@ class AccessContextManagerResource(googleapiclient.discovery.Resource):
self,
*,
name: str,
filter: str = ...,
pageSize: int = ...,
pageToken: str = ...,
returnPartialSuccess: bool = ...,
filter: str | None = ...,
pageSize: int | None = ...,
pageToken: str | None = ...,
returnPartialSuccess: bool | None = ...,
**kwargs: typing.Any,
) -> ListOperationsResponseHttpRequest: ...
def list_next(
Expand Down Expand Up @@ -261,8 +263,8 @@ class AccessContextManagerResource(googleapiclient.discovery.Resource):
self,
*,
parent: str,
pageSize: int = ...,
pageToken: str = ...,
pageSize: int | None = ...,
pageToken: str | None = ...,
**kwargs: typing.Any,
) -> ListGcpUserAccessBindingsResponseHttpRequest: ...
def list_next(
Expand All @@ -275,8 +277,8 @@ class AccessContextManagerResource(googleapiclient.discovery.Resource):
*,
name: str,
body: GcpUserAccessBinding = ...,
append: bool = ...,
updateMask: str = ...,
append: bool | None = ...,
updateMask: str | None = ...,
**kwargs: typing.Any,
) -> OperationHttpRequest: ...

Expand All @@ -285,7 +287,11 @@ class AccessContextManagerResource(googleapiclient.discovery.Resource):
@typing.type_check_only
class PermissionsResource(googleapiclient.discovery.Resource):
def list(
self, *, pageSize: int = ..., pageToken: str = ..., **kwargs: typing.Any
self,
*,
pageSize: int | None = ...,
pageToken: str | None = ...,
**kwargs: typing.Any,
) -> ListSupportedPermissionsResponseHttpRequest: ...
def list_next(
self,
Expand All @@ -299,7 +305,11 @@ class AccessContextManagerResource(googleapiclient.discovery.Resource):
self, *, name: str, **kwargs: typing.Any
) -> SupportedServiceHttpRequest: ...
def list(
self, *, pageSize: int = ..., pageToken: str = ..., **kwargs: typing.Any
self,
*,
pageSize: int | None = ...,
pageToken: str | None = ...,
**kwargs: typing.Any,
) -> ListSupportedServicesResponseHttpRequest: ...
def list_next(
self,
Expand Down
26 changes: 14 additions & 12 deletions googleapiclient-stubs/_apis/accesscontextmanager/v1beta/resources.pyi
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,8 @@ class AccessContextManagerResource(googleapiclient.discovery.Resource):
name: str,
accessLevelFormat: typing_extensions.Literal[
"LEVEL_FORMAT_UNSPECIFIED", "AS_DEFINED", "CEL"
] = ...,
]
| None = ...,
**kwargs: typing.Any,
) -> AccessLevelHttpRequest: ...
def list(
Expand All @@ -38,9 +39,10 @@ class AccessContextManagerResource(googleapiclient.discovery.Resource):
parent: str,
accessLevelFormat: typing_extensions.Literal[
"LEVEL_FORMAT_UNSPECIFIED", "AS_DEFINED", "CEL"
] = ...,
pageSize: int = ...,
pageToken: str = ...,
]
| None = ...,
pageSize: int | None = ...,
pageToken: str | None = ...,
**kwargs: typing.Any,
) -> ListAccessLevelsResponseHttpRequest: ...
def list_next(
Expand All @@ -53,7 +55,7 @@ class AccessContextManagerResource(googleapiclient.discovery.Resource):
*,
name: str,
body: AccessLevel = ...,
updateMask: str = ...,
updateMask: str | None = ...,
**kwargs: typing.Any,
) -> OperationHttpRequest: ...

Expand All @@ -72,8 +74,8 @@ class AccessContextManagerResource(googleapiclient.discovery.Resource):
self,
*,
parent: str,
pageSize: int = ...,
pageToken: str = ...,
pageSize: int | None = ...,
pageToken: str | None = ...,
**kwargs: typing.Any,
) -> ListServicePerimetersResponseHttpRequest: ...
def list_next(
Expand All @@ -86,7 +88,7 @@ class AccessContextManagerResource(googleapiclient.discovery.Resource):
*,
name: str,
body: ServicePerimeter = ...,
updateMask: str = ...,
updateMask: str | None = ...,
**kwargs: typing.Any,
) -> OperationHttpRequest: ...

Expand All @@ -102,9 +104,9 @@ class AccessContextManagerResource(googleapiclient.discovery.Resource):
def list(
self,
*,
pageSize: int = ...,
pageToken: str = ...,
parent: str = ...,
pageSize: int | None = ...,
pageToken: str | None = ...,
parent: str | None = ...,
**kwargs: typing.Any,
) -> ListAccessPoliciesResponseHttpRequest: ...
def list_next(
Expand All @@ -117,7 +119,7 @@ class AccessContextManagerResource(googleapiclient.discovery.Resource):
*,
name: str,
body: AccessPolicy = ...,
updateMask: str = ...,
updateMask: str | None = ...,
**kwargs: typing.Any,
) -> OperationHttpRequest: ...
def accessLevels(self) -> AccessLevelsResource: ...
Expand Down
7 changes: 4 additions & 3 deletions googleapiclient-stubs/_apis/adexchangebuyer/v1_2/resources.pyi
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -35,11 +35,12 @@ class AdExchangeBuyerResource(googleapiclient.discovery.Resource):
def list(
self,
*,
maxResults: int = ...,
pageToken: str = ...,
maxResults: int | None = ...,
pageToken: str | None = ...,
statusFilter: typing_extensions.Literal[
"approved", "disapproved", "not_checked"
] = ...,
]
| None = ...,
**kwargs: typing.Any,
) -> CreativesListHttpRequest: ...
def list_next(
Expand Down
15 changes: 8 additions & 7 deletions googleapiclient-stubs/_apis/adexchangebuyer/v1_3/resources.pyi
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -64,13 +64,14 @@ class AdExchangeBuyerResource(googleapiclient.discovery.Resource):
def list(
self,
*,
accountId: int | _list[int] = ...,
buyerCreativeId: str | _list[str] = ...,
maxResults: int = ...,
pageToken: str = ...,
accountId: int | _list[int] | None = ...,
buyerCreativeId: str | _list[str] | None = ...,
maxResults: int | None = ...,
pageToken: str | None = ...,
statusFilter: typing_extensions.Literal[
"approved", "disapproved", "not_checked"
] = ...,
]
| None = ...,
**kwargs: typing.Any,
) -> CreativesListHttpRequest: ...
def list_next(
Expand All @@ -92,8 +93,8 @@ class AdExchangeBuyerResource(googleapiclient.discovery.Resource):
accountId: str,
endDateTime: str,
startDateTime: str,
maxResults: int = ...,
pageToken: str = ...,
maxResults: int | None = ...,
pageToken: str | None = ...,
**kwargs: typing.Any,
) -> PerformanceReportListHttpRequest: ...

Expand Down
Loading
Loading