-
Notifications
You must be signed in to change notification settings - Fork 374
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
feat(gnostats): add the agent #2706
Conversation
aec0d92
to
c51eb3a
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## dev/zivkovicmilos/gnostats #2706 +/- ##
===========================================================
Coverage 60.14% 60.15%
===========================================================
Files 561 561
Lines 74999 74999
===========================================================
+ Hits 45111 45113 +2
+ Misses 26509 26507 -2
Partials 3379 3379
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks great 💯
There are just a few minor things to change, to make it perfect 🙂
Some areas can be simplified, and we can use different RPC endpoints on the client which would be more performant
86278f5
to
4720128
Compare
4720128
to
77fa392
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you for applying the fixes 🙏
This PR introduces the gnostats agent that acts as a bridge between the Gno node RPC endpoint and gnostats Hub.
Part of #2666.
Contributors' checklist...
BREAKING CHANGE: xxx
message was included in the description