Skip to content

feat: add operationIds to the openAPI spec#47

Merged
FrenchGithubUser merged 1 commit into
mainfrom
tt/operation-ids
Jun 17, 2026
Merged

feat: add operationIds to the openAPI spec#47
FrenchGithubUser merged 1 commit into
mainfrom
tt/operation-ids

Conversation

@FrenchGithubUser

Copy link
Copy Markdown
Member

@FrenchGithubUser
FrenchGithubUser marked this pull request as ready for review June 17, 2026 08:34
@FrenchGithubUser
FrenchGithubUser requested a review from a team as a code owner June 17, 2026 08:34
Copilot AI review requested due to automatic review settings June 17, 2026 08:34

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the project’s OpenAPI specification (SYN-72) to include explicit operationId values for the managed-room endpoints, improving client generation and API operation addressing.

Changes:

  • Added operationId: createManagedRoom to the create-room endpoint.
  • Added operationId: listManagedRooms to the list-rooms endpoint.
  • Added operationId: updateManagedRoomGroups to the update-groups endpoint.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@codecov

codecov Bot commented Jun 17, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.27%. Comparing base (7174183) to head (5c4cc24).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #47   +/-   ##
=======================================
  Coverage   93.27%   93.27%           
=======================================
  Files          10       10           
  Lines         877      877           
  Branches      131      131           
=======================================
  Hits          818      818           
  Misses         34       34           
  Partials       25       25           

Continue to review full report in Codecov by Harness.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7174183...5c4cc24. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jason-famedly jason-famedly left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Seems legit, and harmless if it's not. LGTM

@FrenchGithubUser
FrenchGithubUser added this pull request to the merge queue Jun 17, 2026
Merged via the queue into main with commit 0e176cf Jun 17, 2026
11 checks passed
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.

3 participants