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

Add oldValue to operation when the rationalize is enabled #14

Merged
merged 1 commit into from
Jun 26, 2023

Conversation

woostundy
Copy link
Contributor

We also need to show the old value when rationalization is enabled.
It's helpful in some sense in detail to compare.

@codecov-commenter
Copy link

codecov-commenter commented Jun 26, 2023

Codecov Report

Merging #14 (7ecf313) into master (09a9689) will increase coverage by 0.00%.
The diff coverage is 100.00%.

@@           Coverage Diff           @@
##           master      #14   +/-   ##
=======================================
  Coverage   97.42%   97.43%           
=======================================
  Files           8        8           
  Lines         778      779    +1     
=======================================
+ Hits          758      759    +1     
  Misses         13       13           
  Partials        7        7           
Impacted Files Coverage Δ
differ.go 99.07% <100.00%> (+<0.01%) ⬆️

@wI2L wI2L merged commit c85fb8c into wI2L:master Jun 26, 2023
@wI2L
Copy link
Owner

wI2L commented Jun 26, 2023

@woostundy I'll release a new version in a week or two, I have pending changes on my side, and I'd rather not push a new tag for just this little change.

@woostundy
Copy link
Contributor Author

@woostundy I'll release a new version in a week or two, I have pending changes on my side, and I'd rather not push a new tag for just this little change.

It's OK, just follow your plan.

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