-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
fix(patch): consider escaped characters when applying JSON patches #10717
fix(patch): consider escaped characters when applying JSON patches #10717
Conversation
2345821
to
0c83915
Compare
0c83915
to
33142d1
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
After changes, if CI is passing LGTM!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- entity-registry/src/main/java/com/linkedin/metadata/aspect/patch/template/TemplateUtil.java (1 hunks)
Files skipped from review as they are similar to previous changes (1)
- entity-registry/src/main/java/com/linkedin/metadata/aspect/patch/template/TemplateUtil.java
Do you mind running Thanks! |
6bc5904
to
c136896
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 0
Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Files selected for processing (1)
- entity-registry/src/test/java/com/linkedin/metadata/aspect/patch/template/UpstreamLineageTemplateTest.java (2 hunks)
Files skipped from review as they are similar to previous changes (1)
- entity-registry/src/test/java/com/linkedin/metadata/aspect/patch/template/UpstreamLineageTemplateTest.java
There was a test failure for unrelated code updating with master and trying again |
…10717) Co-authored-by: John Joyce <john@acryl.io>
…atahub-project#10717) Co-authored-by: John Joyce <john@acryl.io>
I have raised an issue for this: #10716
Checklist
Summary by CodeRabbit
New Features
Tests