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

Conversation

@HCastano
Copy link
Contributor

No description provided.

@HCastano HCastano added A0-please_review Pull request needs code review. A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). labels Oct 22, 2019
@HCastano HCastano requested a review from jimpo October 22, 2019 09:30
@parity-cla-bot
Copy link

It looks like @HCastano signed our Contributor License Agreement. 👍

Many thanks,

Parity Technologies CLA Bot

Copy link
Contributor

@andresilva andresilva left a comment

Choose a reason for hiding this comment

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

lgtm other than the unnecessary dependency. but why didn't you push this directly to your branch?

@HCastano
Copy link
Contributor Author

lgtm other than the unnecessary dependency. but why didn't you push this directly to your branch?

I wanted to ping Jim about this without DM-ing him about the changes.

@HCastano HCastano merged commit b3a089d into paritytech:hc-jp-bridge-module Oct 23, 2019
@HCastano HCastano deleted the hc-update-tests branch October 23, 2019 12:55
HCastano added a commit to HCastano/substrate that referenced this pull request Nov 11, 2019
…tytech#3874)

* Make tests work after the changes introduced in paritytech#3793

* Remove unneccessary import
HCastano added a commit that referenced this pull request Dec 12, 2019
* Make tests work after the changes introduced in #3793

* Remove unneccessary import
HCastano added a commit that referenced this pull request Jan 10, 2020
* Make tests work after the changes introduced in #3793

* Remove unneccessary import
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. A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update).

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants