Skip to content

Align commnalities#13

Merged
chinaunicomyangfan merged 3 commits intocamaraproject:mainfrom
chinaunicomyangfan:AlignCommnalities
Jul 10, 2025
Merged

Align commnalities#13
chinaunicomyangfan merged 3 commits intocamaraproject:mainfrom
chinaunicomyangfan:AlignCommnalities

Conversation

@chinaunicomyangfan
Copy link
Contributor

What type of PR is this?

Add one of the following kinds:

  • enhancement/feature

What this PR does / why we need it:

change yaml file to align commonalities v0.6.0

Remove AUTHENTICATION_REQUIRED error code
Update x-correlator format
Add text on undocumented errors to OAS definition

Which issue(s) this PR fixes:

Fixes #10

Special notes for reviewers:

Changelog input

 release-note

Additional documentation

This section can be blank.

docs

ChuanyuChen
ChuanyuChen previously approved these changes Jun 28, 2025
Copy link

@ChuanyuChen ChuanyuChen left a comment

Choose a reason for hiding this comment

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

looks good

@chinaunicomyangfan chinaunicomyangfan dismissed ChuanyuChen’s stale review June 28, 2025 01:33

The merge-base changed after approval.

ChuanyuChen
ChuanyuChen previously approved these changes Jun 28, 2025
Copy link

@ChuanyuChen ChuanyuChen left a comment

Choose a reason for hiding this comment

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

looks good.

@chinaunicomyangfan chinaunicomyangfan dismissed ChuanyuChen’s stale review June 28, 2025 01:44

The merge-base changed after approval.

@ChuanyuChen ChuanyuChen self-requested a review July 9, 2025 02:40
ChuanyuChen
ChuanyuChen previously approved these changes Jul 9, 2025
Copy link

@ChuanyuChen ChuanyuChen left a comment

Choose a reason for hiding this comment

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

looks Good to me

@chinaunicomyangfan chinaunicomyangfan dismissed ChuanyuChen’s stale review July 9, 2025 02:40

The merge-base changed after approval.

@ChuanyuChen ChuanyuChen self-requested a review July 9, 2025 02:43
ChuanyuChen
ChuanyuChen previously approved these changes Jul 9, 2025
Copy link

@ChuanyuChen ChuanyuChen left a comment

Choose a reason for hiding this comment

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

looks ok.

@chinaunicomyangfan chinaunicomyangfan dismissed ChuanyuChen’s stale review July 9, 2025 02:45

The merge-base changed after approval.

ClownsAlbert
ClownsAlbert previously approved these changes Jul 9, 2025
Copy link

@ClownsAlbert ClownsAlbert left a comment

Choose a reason for hiding this comment

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

ok

@chinaunicomyangfan chinaunicomyangfan dismissed ClownsAlbert’s stale review July 9, 2025 04:29

The merge-base changed after approval.

@ChuanyuChen ChuanyuChen self-requested a review July 10, 2025 07:36
Copy link

@ChuanyuChen ChuanyuChen left a comment

Choose a reason for hiding this comment

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

looks good.

@chinaunicomyangfan chinaunicomyangfan merged commit d60b5ac into camaraproject:main Jul 10, 2025
2 checks passed
@chinaunicomyangfan chinaunicomyangfan deleted the AlignCommnalities branch March 2, 2026 08:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Commonalities alignment

3 participants