Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

W 12689960 New field at messages: MessageId #1938

Merged
merged 2 commits into from
Feb 27, 2024
Merged

W 12689960 New field at messages: MessageId #1938

merged 2 commits into from
Feb 27, 2024

Conversation

damianpedra
Copy link
Contributor

New field at messages: MessageId

parseMessageId(map, message)
}
message
// map.key("messageId", MessageModel.MessageId in message)
Copy link
Contributor

Choose a reason for hiding this comment

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

Remove commented lines

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixex

val messageId = entry.value.toString()
if (!ctx.registerMessageId(messageId))
ctx.eh.violation(
ParserSideValidations.DuplicatedMessageId, message, s"duplicado $messageId", entry.value.location
Copy link
Contributor

Choose a reason for hiding this comment

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

Fix error message "duplicado"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

validate("double-references/valid-ref-to-parameter-with-ref.yaml")
}

test("Valid ref to bindings with another ref") {
Copy link
Contributor

Choose a reason for hiding this comment

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

Why all these deleted lines?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed

@damianpedra damianpedra force-pushed the W-12689960 branch 3 times, most recently from 8f6bef0 to f7ad481 Compare February 27, 2024 18:33
@damianpedra damianpedra merged commit 76c8fa5 into develop Feb 27, 2024
6 checks passed
@damianpedra damianpedra deleted the W-12689960 branch February 27, 2024 19:25
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.

2 participants