-
Notifications
You must be signed in to change notification settings - Fork 246
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
2476.add json welcome page.0 #430
2476.add json welcome page.0 #430
Conversation
This code looks fine to me too. It might be nice to squash some of this down to fewer commits, but I'm going to practice not being uptight about that :) |
scared about duplicate nodeids in the data getting lost, switch to a list
flat is better than nested
7ca583e
to
1f16a17
Compare
Codecov Report
@@ Coverage Diff @@
## master #430 +/- ##
==========================================
- Coverage 88.02% 88.02% -0.01%
==========================================
Files 147 147
Lines 27820 27836 +16
Branches 3972 3974 +2
==========================================
+ Hits 24489 24503 +14
- Misses 2617 2619 +2
Partials 714 714
Continue to review full report at Codecov.
|
It provides information more correctly obtained from get_connection_status().
ok, looks good, merging now |
This is an update of #399
It's stacked on #429. Review that first (and this branch have history rewritten depending on what happens there).