Skip to content

fix On-Call spec #2572

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

Merged

Conversation

api-clients-generation-pipeline[bot]
Copy link
Contributor

@api-clients-generation-pipeline api-clients-generation-pipeline bot requested review from a team as code owners May 20, 2025 13:11
@api-clients-generation-pipeline api-clients-generation-pipeline bot force-pushed the datadog-api-spec/generated/3892 branch 9 times, most recently from f782499 to 600ff9b Compare May 22, 2025 08:50
def __init__(
self_, targets: Union[EscalationTarget, TeamTarget, UserTarget, ScheduleTarget, UnsetType] = unset, **kwargs
):
def __init__(self_, targets: Union[EscalationTargets, UnsetType] = unset, **kwargs):

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔴 Code Quality Violation

Suggested change
def __init__(self_, targets: Union[EscalationTargets, UnsetType] = unset, **kwargs):
def __init__(self, targets: Union[EscalationTargets, UnsetType] = unset, **kwargs):
first parameter of a class function should be self (...read more)

In a class method (that is not a class method nor a static method), the first argument must be self by convention.

Learn More

View in Datadog  Leave us feedback  Documentation

time_zone: str,
tags: Union[List[str], UnsetType] = unset,
**kwargs,
self_, layers: List[ScheduleCreateRequestDataAttributesLayersItems], name: str, time_zone: str, **kwargs

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔴 Code Quality Violation

Suggested change
self_, layers: List[ScheduleCreateRequestDataAttributesLayersItems], name: str, time_zone: str, **kwargs
self, layers: List[ScheduleCreateRequestDataAttributesLayersItems], name: str, time_zone: str, **kwargs
first parameter of a class function should be self (...read more)

In a class method (that is not a class method nor a static method), the first argument must be self by convention.

Learn More

View in Datadog  Leave us feedback  Documentation

time_zone: str,
tags: Union[List[str], UnsetType] = unset,
**kwargs,
self_, layers: List[ScheduleUpdateRequestDataAttributesLayersItems], name: str, time_zone: str, **kwargs

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔴 Code Quality Violation

Suggested change
self_, layers: List[ScheduleUpdateRequestDataAttributesLayersItems], name: str, time_zone: str, **kwargs
self, layers: List[ScheduleUpdateRequestDataAttributesLayersItems], name: str, time_zone: str, **kwargs
first parameter of a class function should be self (...read more)

In a class method (that is not a class method nor a static method), the first argument must be self by convention.

Learn More

View in Datadog  Leave us feedback  Documentation

}

def __init__(
self_,

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔴 Code Quality Violation

Suggested change
self_,
self,
first parameter of a class function should be self (...read more)

In a class method (that is not a class method nor a static method), the first argument must be self by convention.

Learn More

View in Datadog  Leave us feedback  Documentation

time_zone: Union[str, UnsetType] = unset,
**kwargs,
):
def __init__(self_, name: Union[str, UnsetType] = unset, time_zone: Union[str, UnsetType] = unset, **kwargs):

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔴 Code Quality Violation

Suggested change
def __init__(self_, name: Union[str, UnsetType] = unset, time_zone: Union[str, UnsetType] = unset, **kwargs):
def __init__(self, name: Union[str, UnsetType] = unset, time_zone: Union[str, UnsetType] = unset, **kwargs):
first parameter of a class function should be self (...read more)

In a class method (that is not a class method nor a static method), the first argument must be self by convention.

Learn More

View in Datadog  Leave us feedback  Documentation

}

attribute_map = {
"identifier": "identifier",
"type": "type",
}

def __init__(self_, identifier: Union[str, UnsetType] = unset, type: Union[str, UnsetType] = unset, **kwargs):
def __init__(
self_, identifier: Union[str, UnsetType] = unset, type: Union[OnCallPageTargetType, UnsetType] = unset, **kwargs

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🔴 Code Quality Violation

Suggested change
self_, identifier: Union[str, UnsetType] = unset, type: Union[OnCallPageTargetType, UnsetType] = unset, **kwargs
self, identifier: Union[str, UnsetType] = unset, type: Union[OnCallPageTargetType, UnsetType] = unset, **kwargs
first parameter of a class function should be self (...read more)

In a class method (that is not a class method nor a static method), the first argument must be self by convention.

Learn More

View in Datadog  Leave us feedback  Documentation

@api-clients-generation-pipeline api-clients-generation-pipeline bot force-pushed the datadog-api-spec/generated/3892 branch 2 times, most recently from b4429ae to a0faecd Compare May 23, 2025 17:21
@api-clients-generation-pipeline api-clients-generation-pipeline bot force-pushed the datadog-api-spec/generated/3892 branch from a0faecd to 4710cb0 Compare May 23, 2025 19:26
@api-clients-generation-pipeline api-clients-generation-pipeline bot merged commit debc5df into master May 26, 2025
12 checks passed
@api-clients-generation-pipeline api-clients-generation-pipeline bot deleted the datadog-api-spec/generated/3892 branch May 26, 2025 11:47
github-actions bot pushed a commit that referenced this pull request May 26, 2025
Co-authored-by: ci.datadog-api-spec <packages@datadoghq.com> debc5df
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant