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

add a new unit test for extrinsic pool #611

Merged
merged 1 commit into from
Aug 28, 2018

Conversation

guanqun
Copy link

@guanqun guanqun commented Aug 26, 2018

No description provided.

Copy link
Contributor

@tomusdrw tomusdrw left a comment

Choose a reason for hiding this comment

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

👍

@guanqun
Copy link
Author

guanqun commented Aug 27, 2018

Looks like our CI is not robust enough

error: could not rename component file from '/root/.rustup/toolchains/nightly-x86_64-unknown-linux-gnu/lib/rustlib/etc' to '/root/.rustup/tmp/hm2aoptql56vt5g6_dir/bk'
info: caused by: Invalid cross-device link (os error 18)

Do we have a way to ask CI to retry instead of rebasing my branch and pushing again?

@pepyakin
Copy link
Contributor

pepyakin commented Aug 27, 2018

@guanqun It looks to me that mere retrying won't help. I've asked our CI wizard to take a look!

@gavofyork gavofyork merged commit 8e63e90 into paritytech:master Aug 28, 2018
@guanqun guanqun deleted the gq-add-unit-test branch August 28, 2018 10:14
dvdplm added a commit that referenced this pull request Aug 29, 2018
* master:
  Contracts: Per block gas limit (#506)
  Make sure to ban invalid transactions. (#615) (#620)
  Forward-port BFT fixes from v0.2 and restructure agreement cancelling (#619)
  Allow specifying listening multiaddresses (#577)
  Introduce Runtime Events (#607)
  update substrate/extrinsic-pool (#616)
  add a new unit test for extrinsic pool (#611)
  set the current repo in Cargo.toml (#610)
  add cli for purge chain (#609)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants