This repository was archived by the owner on Nov 15, 2023. It is now read-only.
-
Notifications
You must be signed in to change notification settings - Fork 371
Use JustifiedBlockAnnounceValidator for parachain block announce validator #96
Merged
Merged
Changes from 1 commit
Commits
Show all changes
60 commits
Select commit
Hold shift + click to select a range
655dc45
Initial commit
cecton 9a47c09
Added subcommand to run polkadot
cecton 32e81cf
Add command validation-worker
cecton df0f832
Add storage_append to runtime
cecton 892d057
WIP
cecton 41f50ee
WIP
cecton 214e50d
WIP
cecton cd131ff
WIP
cecton 3212554
clean exit if possible
cecton 0286a88
wip
cecton d38edea
WIP
cecton 3d162e0
WIP
cecton 62b544e
WIP
cecton 17887fc
.editorconfig
cecton a0eb356
register working
cecton 9961be4
WIP
cecton baa4357
clean-up
cecton fc408e4
Rename ProcessCleanUp to ChildHelper
cecton 22b64de
More clean-up
cecton a978970
Runs integration test separately
cecton ee3832a
Merge commit 3acb6d0d835c3589f50bf0911729baaea41981fc (no conflict)
cecton 1267f80
Merge commit c163d8702b27e542caebd0df585d45ec761edc2a (conflicts)
cecton 368a582
Simplify toml
cecton de2b666
Merge commit e810cf3b5ba4b6b1a2848bc57a6302146fc54d78 (conflicts)
cecton a17dd34
Merge commit d6ab13c00f555674212b0c86620fdb504d8cf505 (no conflict)
cecton 31768ce
Initial commit
cecton 68b0dcf
Update substrate & polkadot branch
cecton e2f3fdc
Update test/parachain/tests/integration_test.rs
cecton 0361e14
Split polkadot service creation from service building/running
cecton cde8935
Applied suggestions
cecton 145ec00
Renamed _child to _helper
cecton 0d21e75
Use JustifiedBlockAnnounceValidator
cecton 671d5f9
Update test/parachain/tests/integration_test.rs
cecton a5c89b0
Make a function for generate_genesis_state
cecton 9697b62
Move std::io::Read import
cecton a300ea9
wait_for_tcp indefinitely + remove 10s wait for relay blocks
cecton 87489c5
Implement max time allowed for the test
cecton 6f0075a
rpc call to monitor new blocks
cecton 9c7af42
Removed unused helpers
cecton 19b7942
Update test/parachain/tests/integration_test.rs
cecton 6eca4f0
Not sure why I made a helper for this in the first place...
cecton 6e43fd4
WIP
cecton 2564cce
cleanup
cecton 542eb35
CLEANUP
cecton ddb111f
Merge branch 'cecton-int-test' into cecton-use-custom-block-announce-…
cecton d985af8
WIP
cecton e4fb6aa
Apply suggestions
cecton bf68f24
Fix p2p ports
cecton 8c24090
Merge branch 'cecton-int-test' into cecton-use-custom-block-announce-…
cecton a633fdd
Add test in int test scenario
cecton 865542d
CLEANUP
cecton 56929d6
CLEANUP
cecton 0b1f21b
Merge commit e183d8bd30514d941586bb9bb6decce322572fca (no conflict)
cecton c9f95a3
Merge commit 0a7cde66102efe59f4041e5fe5603eace6443328 (conflicts)
cecton e7fe457
Refactor
cecton bb89a33
Removes some unwrap & introduce parking_lot
cecton 9cc6f21
Replace match by expect
cecton 5a257d6
Update network/src/lib.rs
cecton 934fd5c
Add doc
cecton 21dba8c
Reduce comment
cecton File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
CLEANUP
Forked at: d6ab13c Parent branch: origin/master
- Loading branch information
commit 542eb3511d501f1922948607e8fe6a02b8d804bb
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Docs and especially why we do this! + an issue that we need to fix this.
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.
done
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.
You should create an issue for this. And this should contain this info.
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.
Ok! Sorry, created #104
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.
done