Repository navigation
APIS-6645: resolve bugs and typos in transformSchema - #162
Conversation
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates transformSchema.json by adding type to the required properties of the current time schema, specifying the type for the format property, fixing a typo in a description, and renaming the format property to delimiter. Feedback highlights a potential schema validation issue where the required array preceding the renamed delimiter property likely still references the old format property and needs to be updated.
| "additionalProperties": false, | ||
| "properties": { | ||
| "format": { | ||
| "delimiter": { |
There was a problem hiding this comment.
The property name has been changed from format to delimiter to match the Go implementation for toCamelCase. However, the required array immediately preceding this block (ending on line 344) likely still lists format as a required property. If so, this will cause schema validation to fail because format is no longer defined and additionalProperties is set to false. Please update the required array to require delimiter instead of format.
![orca-security-us[bot]](https://avatars.githubusercontent.com/in/276250?s=60&v=4){kind=small}
There was a problem hiding this comment.
Orca Security Scan Summary
| Status | Check | Issues by priority | |
|---|---|---|---|
 Passed |
Infrastructure as Code |  0  0  0  0 |
View in Orca |
| Passed |
SAST | 0 0 0 0 |
View in Orca |
| Passed |
Secrets | 0 0 0 0 |
View in Orca |
| Passed |
Vulnerabilities | 0 0 0 0 |
View in Orca |
|
coverage: 32.634%. remained the same — APIS-6645 into master |
No description provided.