twitter-archive / diffy

Find potential bugs in your services with Diffy
https://twitter.com/diffyproject
Apache License 2.0
3.83k stars 368 forks source link

Sorting JSON arrays by toString removed #51

Open x87-va opened 7 years ago

x87-va commented 7 years ago

When lifting nested JSON arrays diffy looses original ordering and generates diffs.

codecov-io commented 7 years ago

Current coverage is 35.02% (diff: 100%)

Merging #51 into master will decrease coverage by 0.08%

@@             master        #51   diff @@
==========================================
  Files            29         29          
  Lines           789        788     -1   
  Methods         769        753    -16   
  Messages          0          0          
  Branches         20         35    +15   
==========================================
- Hits            277        276     -1   
  Misses          512        512          
  Partials          0          0          

Powered by Codecov. Last update 2077282...8d94b76

CLAassistant commented 5 years ago

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.


Виталий Левашов seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.