Skip to content

fix: add reference to CAMARA_common.yaml schemas#28

Merged
JoseMConde merged 5 commits into
camaraproject:mainfrom
DLondonoD:dev-comm-review-2
May 28, 2026
Merged

fix: add reference to CAMARA_common.yaml schemas#28
JoseMConde merged 5 commits into
camaraproject:mainfrom
DLondonoD:dev-comm-review-2

Conversation

@DLondonoD
Copy link
Copy Markdown
Contributor

What type of PR is this?

  • correction

What this PR does / why we need it:

To avoid duplication, references to CAMARA_common.yaml are added for x-correlator, ErrorInfo, and errors.

Which issue(s) this PR fixes:

Identified by Sergi in #27 (comment)

Special notes for reviewers:

Changelog input

 release-note

Additional documentation

This section can be blank.

docs

JoseMConde
JoseMConde previously approved these changes May 27, 2026
Copy link
Copy Markdown
Contributor

@JoseMConde JoseMConde left a comment

Choose a reason for hiding this comment

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

LGTM

seralogar
seralogar previously approved these changes May 27, 2026
@DLondonoD DLondonoD dismissed stale reviews from seralogar and JoseMConde May 28, 2026 09:13

The merge-base changed after approval.

seralogar
seralogar previously approved these changes May 28, 2026
@JoseMConde JoseMConde self-requested a review May 28, 2026 09:17
JoseMConde
JoseMConde previously approved these changes May 28, 2026
Copy link
Copy Markdown
Contributor

@JoseMConde JoseMConde left a comment

Choose a reason for hiding this comment

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

LGTM

@DLondonoD DLondonoD dismissed stale reviews from JoseMConde and seralogar May 28, 2026 09:19

The merge-base changed after approval.

@DLondonoD DLondonoD force-pushed the dev-comm-review-2 branch from 7cb84c2 to fe09040 Compare May 28, 2026 09:41
@JoseMConde JoseMConde merged commit dd87189 into camaraproject:main May 28, 2026
2 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