-
Notifications
You must be signed in to change notification settings - Fork 6
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
Make JSON Diff instead of what is there now #68
base: master
Are you sure you want to change the base?
Conversation
Packet Coverage Graph
Packet Coverage Stats
|
Packet Coverage Graph
Packet Coverage Stats
|
Packet Coverage Graph
Packet Coverage Stats
|
Packet Coverage Graph
| Packet Name || Packet Coverage Stats
|
Packet Coverage Graph
Packet Coverage Stats
|
Packet Coverage Graph
Packet Coverage Stats
|
Packet Coverage Graph
Packet Coverage Stats
|
Packet Coverage Graph
Packet Coverage Stats
|
Packet Coverage Graph
| Packet Name || Packet Coverage Stats
|
Packet Coverage Graph
| Packet Name || Packet Coverage Stats
|
1 similar comment
Packet Coverage Graph
| Packet Name || Packet Coverage Stats
|
Packet Coverage Graph
| Packet Name || Packet Coverage Stats
|
Packet Coverage Graph
| Packet Name || Packet Coverage Stats
|
Packet Coverage Graph
Packet Coverage Stats
|
Packet Coverage Graph
Packet Coverage Stats
|
Packet Coverage Graph
Packet Coverage Stats
|
Packet Coverage Graph
Packet Coverage Stats
|
Example of what a new diff will look like: u9g/test1#3 (comment) , in my opinion it's a lot more readable and the unimportant data like versions that have no changes is not shown , I did remove the % completion, but I'll add that back after this gets merged |
Packet Coverage Graph
Packet Coverage Stats
|
Packet Coverage Graph
| Packet Name || Packet Coverage Stats
|
Packet Coverage Graph
Packet Coverage Stats
|
@u9g should we merge ? |
@u9g should I merge or close ? |
Packet Coverage Graph
Packet Coverage Stats
|
Packet Coverage Graph
Packet Coverage Stats
|
Here's how the output should look assuming nothing breaks (it will tho ;)) u9g/test1#3 (comment) |
Packet Coverage Graph
Packet Coverage Stats
|
hey @u9g do you remember the status here? should we try to merge this or close it ? |
Packet Coverage Graph
Packet Coverage Stats
|
seems to run, I'll need to look at the code to understand what this does |
this pr adds a bench/ folder to the repo to report what packets can be gotten using the packet dumper, which is also auto-remade on every push to master