Skip to content

feat:Allow null for components.properties.sha in OpenAPI spec#181

Merged
github-actions[bot] merged 1 commit intomainfrom
bot/update-openapi_202509160608
Sep 16, 2025
Merged

feat:Allow null for components.properties.sha in OpenAPI spec#181
github-actions[bot] merged 1 commit intomainfrom
bot/update-openapi_202509160608

Conversation

@HavenDV
Copy link
Owner

@HavenDV HavenDV commented Sep 16, 2025

Summary by CodeRabbit

  • Bug Fixes
    • Updated the OpenAPI specification to allow the sha property to be null in addition to a string. This ensures client SDKs and validators accept responses where sha may be absent and prevents schema validation errors.

@coderabbitai
Copy link

coderabbitai bot commented Sep 16, 2025

Caution

Review failed

The pull request is closed.

Walkthrough

The OpenAPI specification was updated so that the sha property under components.properties.sha now allows nulls by adding nullable: true alongside type: string. No other properties or files were changed.

Changes

Cohort / File(s) Summary
OpenAPI schema update
src/libs/GitHub/openapi.yaml
Added nullable: true to components.properties.sha, allowing sha to be string or null; example and other schema parts unchanged.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

I twitch my nose at schema’s call,
A tiny hop—now nulls can fall.
The SHA may hide or plainly sing,
Our YAML yawns, a simple thing.
Thump-thump! I stamp, review is through—
One carrot’s tweak, and off I zoom. 🥕🐇

✨ Finishing touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch bot/update-openapi_202509160608

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 5201363 and 89a7643.

⛔ Files ignored due to path filters (1)
  • src/libs/GitHub/Generated/GitHub.Models.DiffEntry.g.cs is excluded by !**/generated/**
📒 Files selected for processing (1)
  • src/libs/GitHub/openapi.yaml (1 hunks)

Warning

Tools execution failed with the following error:

Failed to run tools: 14 UNAVAILABLE: read ECONNRESET


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions github-actions bot merged commit cfd662e into main Sep 16, 2025
1 of 3 checks passed
@coderabbitai coderabbitai bot changed the title feat:@coderabbitai feat:Allow null for components.properties.sha in OpenAPI spec Sep 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant