Add Datasets API to Open API Spec #2492
Conversation
![api-clients-generation-pipeline[bot]](https://avatars.githubusercontent.com/u/365230?s=60&v=4){kind=small}
api-clients-generation-pipeline
bot
changed the title
api-clients-generation-pipeline
bot
force-pushed
the
datadog-api-spec/generated/3999
branch
4 times, most recently
from
33510f1 to
2ca8de7
Compare
| * This is a holder for any undeclared properties as specified with | ||
| * the 'additionalProperties' keyword in the OAS document. | ||
| */ | ||
| "additionalProperties"?: { [key: string]: any }; |
There was a problem hiding this comment.
⚪ Code Quality Violation
Unexpected any. Specify a different type. (...read more)
Do not use the any type, as it is too broad and can lead to unexpected behavior.
| /** | ||
| * List of product-specific filters. | ||
| */ | ||
| "productFilters": Array<FiltersPerProduct>; |
There was a problem hiding this comment.
⚪ Code Quality Violation
Array type using 'Array' is forbidden. Use 'FiltersPerProduct[]' instead. (...read more)
Chose between: T[] and Array<T>. These types are equivalent and it should be consistent across the code base.
| * This is a holder for any undeclared properties as specified with | ||
| * the 'additionalProperties' keyword in the OAS document. | ||
| */ | ||
| "additionalProperties"?: { [key: string]: any }; |
There was a problem hiding this comment.
⚪ Code Quality Violation
Unexpected any. Specify a different type. (...read more)
Do not use the any type, as it is too broad and can lead to unexpected behavior.
| * This is a holder for any undeclared properties as specified with | ||
| * the 'additionalProperties' keyword in the OAS document. | ||
| */ | ||
| "additionalProperties"?: { [key: string]: any }; |
There was a problem hiding this comment.
⚪ Code Quality Violation
Unexpected any. Specify a different type. (...read more)
Do not use the any type, as it is too broad and can lead to unexpected behavior.
| * This is a holder for any undeclared properties as specified with | ||
| * the 'additionalProperties' keyword in the OAS document. | ||
| */ | ||
| "additionalProperties"?: { [key: string]: any }; |
There was a problem hiding this comment.
⚪ Code Quality Violation
Unexpected any. Specify a different type. (...read more)
Do not use the any type, as it is too broad and can lead to unexpected behavior.
| return FiltersPerProduct.attributeTypeMap; | ||
| } | ||
|
|
||
| public constructor() {} |
There was a problem hiding this comment.
🟠 Code Quality Violation
Avoid useless constructors (...read more)
This rule is designed to flag constructors that either do nothing or only call the super function. These constructors are unnecessary and can be safely removed. In JavaScript, if a class extends another class and does not have a constructor, it automatically calls the super function with all the arguments it receives.
Unnecessary constructors can lead to confusion for other developers who may be reading or maintaining your code. They might spend time trying to figure out why a constructor is there when it doesn't need to be, or they might assume that the constructor is doing something important when it's not. To follow this rule and write good, clean code, you should only write a constructor if it's doing something other than just calling super.
| return DatasetResponseSingle.attributeTypeMap; | ||
| } | ||
|
|
||
| public constructor() {} |
There was a problem hiding this comment.
🟠 Code Quality Violation
Avoid useless constructors (...read more)
This rule is designed to flag constructors that either do nothing or only call the super function. These constructors are unnecessary and can be safely removed. In JavaScript, if a class extends another class and does not have a constructor, it automatically calls the super function with all the arguments it receives.
Unnecessary constructors can lead to confusion for other developers who may be reading or maintaining your code. They might spend time trying to figure out why a constructor is there when it doesn't need to be, or they might assume that the constructor is doing something important when it's not. To follow this rule and write good, clean code, you should only write a constructor if it's doing something other than just calling super.
| return Dataset.attributeTypeMap; | ||
| } | ||
|
|
||
| public constructor() {} |
There was a problem hiding this comment.
🟠 Code Quality Violation
Avoid useless constructors (...read more)
This rule is designed to flag constructors that either do nothing or only call the super function. These constructors are unnecessary and can be safely removed. In JavaScript, if a class extends another class and does not have a constructor, it automatically calls the super function with all the arguments it receives.
Unnecessary constructors can lead to confusion for other developers who may be reading or maintaining your code. They might spend time trying to figure out why a constructor is there when it doesn't need to be, or they might assume that the constructor is doing something important when it's not. To follow this rule and write good, clean code, you should only write a constructor if it's doing something other than just calling super.
| return DatasetCreateRequest.attributeTypeMap; | ||
| } | ||
|
|
||
| public constructor() {} |
There was a problem hiding this comment.
🟠 Code Quality Violation
Avoid useless constructors (...read more)
This rule is designed to flag constructors that either do nothing or only call the super function. These constructors are unnecessary and can be safely removed. In JavaScript, if a class extends another class and does not have a constructor, it automatically calls the super function with all the arguments it receives.
Unnecessary constructors can lead to confusion for other developers who may be reading or maintaining your code. They might spend time trying to figure out why a constructor is there when it doesn't need to be, or they might assume that the constructor is doing something important when it's not. To follow this rule and write good, clean code, you should only write a constructor if it's doing something other than just calling super.
| return DatasetResponseMulti.attributeTypeMap; | ||
| } | ||
|
|
||
| public constructor() {} |
There was a problem hiding this comment.
🟠 Code Quality Violation
Avoid useless constructors (...read more)
This rule is designed to flag constructors that either do nothing or only call the super function. These constructors are unnecessary and can be safely removed. In JavaScript, if a class extends another class and does not have a constructor, it automatically calls the super function with all the arguments it receives.
Unnecessary constructors can lead to confusion for other developers who may be reading or maintaining your code. They might spend time trying to figure out why a constructor is there when it doesn't need to be, or they might assume that the constructor is doing something important when it's not. To follow this rule and write good, clean code, you should only write a constructor if it's doing something other than just calling super.
api-clients-generation-pipeline
bot
force-pushed
the
datadog-api-spec/generated/3999
branch
from
2ca8de7 to
8c7bdf5
Compare
api-clients-generation-pipeline
bot
force-pushed
the
datadog-api-spec/generated/3999
branch
from
8c7bdf5 to
ad6980f
Compare
api-clients-generation-pipeline
bot
deleted the
datadog-api-spec/generated/3999
branch
Co-authored-by: ci.datadog-api-spec <[email protected]> ca9b4ab
See DataDog/datadog-api-spec#3999
Test branch datadog-api-spec/test/datasets-open-api