Skip to content
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.

Sign up for GitHub

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

Jump to bottom

bench-bot: Supporting patching a custom dependency into the branch before benchmark #41

Closed
joao-paulo-parity opened this issue May 12, 2022 · 1 comment · Fixed by #64
Closed

bench-bot: Supporting patching a custom dependency into the branch before benchmark #41

joao-paulo-parity opened this issue May 12, 2022 · 1 comment · Fixed by #64
Assignees
@joao-paulo-parity
Labels
enhancement New feature or request

Comments

@joao-paulo-parity
Copy link
Contributor

joao-paulo-parity commented May 12, 2022 •
edited
Loading

Originally from paritytech/bench-bot#61

If one wants to use a custom Substrate ref for benchmarking a Polkadot PR, allow that through PATCH_SUBSTRATE. e.g. if I want to use PR #11375 I'd say /cmd queue -v PATCH_SUBSTRATE=refs/pull/11375/merge -c bench-bot $ runtime westend-dev pallet_balances. This can be useful for companion PRs because they're not compiling before their dependencies are merged.
@joao-paulo-parity joao-paulo-parity added the enhancement New feature or request label May 12, 2022
@joao-paulo-parity joao-paulo-parity self-assigned this Jul 6, 2022
@joao-paulo-parity joao-paulo-parity mentioned this issue Jul 6, 2022
Closed
@joao-paulo-parity joao-paulo-parity mentioned this issue Jul 13, 2022
Merged
@joao-paulo-parity
Copy link
Contributor Author

Added examples of how it works in #63

@joao-paulo-parity joao-paulo-parity mentioned this issue Jul 13, 2022
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@joao-paulo-parity joao-paulo-parity

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Introduce try-runtime-bot script joao-paulo-parity/pipeline-scripts
1 participant
@joao-paulo-parity