Skip to content
This repository was archived by the owner on Nov 15, 2023. It is now read-only.

Conversation

@athei
Copy link
Member

@athei athei commented Aug 9, 2021

It seems that we are not going to upgrade this dependency but rather wait for jsonrpsee. In the meantime the PRs raised by dependabot are somewhat annoying. This disables them.

@athei athei added A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. labels Aug 9, 2021
@athei athei requested a review from a team as a code owner August 9, 2021 07:14
@athei athei requested review from andresilva and dvdplm August 9, 2021 07:39
Copy link
Contributor

@TriplEight TriplEight left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@bkchr bkchr merged commit f2b3997 into master Aug 9, 2021
@bkchr bkchr deleted the at-ignore-deps branch August 9, 2021 11:01
@athei athei mentioned this pull request Aug 12, 2021
bkchr added a commit that referenced this pull request Aug 12, 2021
ghost pushed a commit that referenced this pull request Aug 13, 2021
* Upgrade jsonrpc to 0.18.0

I think this says all :P

* 🤦

* Fmt etc

* Fix tests

* Fix tests again...

* Better impl

* Revert "Tell dependabot to ignore jsonrpc-* updates (#9518)"

This reverts commit f2b3997.
chevdor pushed a commit that referenced this pull request Sep 7, 2021
* Upgrade jsonrpc to 0.18.0

I think this says all :P

* 🤦

* Fmt etc

* Fix tests

* Fix tests again...

* Better impl

* Revert "Tell dependabot to ignore jsonrpc-* updates (#9518)"

This reverts commit f2b3997.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants