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.

Sign up for GitHub

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

Jump to bottom

ygot.FormatDiff #929

Merged
merged 3 commits into from
Nov 7, 2023
Merged

ygot.FormatDiff #929

merged 3 commits into from
Nov 7, 2023

Conversation

wenovus
Copy link
Collaborator

@wenovus wenovus commented Nov 7, 2023 •
edited
Loading

Sample output:

deleted: /floatval
new/updated /int-val: int_val:10

no diff

@wenovus wenovus requested review from robshakir and dplore November 7, 2023 01:27
@coveralls
Copy link

coveralls commented Nov 7, 2023 •
edited
Loading

Coverage Status

coverage: 89.514% (-0.008%) from 89.522%
when pulling de97eef on diff-format
into d970f97 on master.

robshakir
robshakir approved these changes Nov 7, 2023
View reviewed changes
ygot/diff.go Show resolved Hide resolved
ygot/diff_test.go Show resolved Hide resolved
@wenovus wenovus merged commit 7172fd7 into master Nov 7, 2023
9 checks passed
@wenovus wenovus deleted the diff-format branch November 7, 2023 03:05
@dplore
Copy link
Member

dplore commented Nov 7, 2023

Thanks for this Wen!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@robshakir robshakir robshakir approved these changes

@dplore dplore Awaiting requested review from dplore

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@wenovus @coveralls @dplore @robshakir