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.

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

Support using a separate benchmarks repo for benchmarks #427

Merged
merged 1 commit into from Apr 3, 2023

Conversation

punchagan
Copy link
Contributor

For repositories like ocaml/ocaml, we would like the benchmarks to be maintained potentially by a bigger/different set of users, than the ocaml/ocaml maintainers. This commit adds support for an additional configuration variable that lets us clone an additional repository and run benchmarks from within it.

NOTE: No new benchmark runs would be triggered by any changes to the benchmarks repository, with this setup.

This is a simplistic PR that attempts to implement functionality similar to #370 without the UI hackery.

For repositories like ocaml/ocaml, we would like the benchmarks to be
maintained potentially by a bigger/different set of users, than the ocaml/ocaml
maintainers. This commit adds support for an additional configuration variable
that lets us clone an additional repository and run benchmarks from within it.

NOTE: No new benchmark runs would be triggered by any changes to the benchmarks
repository, with this setup.
Copy link
Contributor

@art-w art-w left a comment

Choose a reason for hiding this comment

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

Nice! Your solution is even easier than I expected!

@ElectreAAS
Copy link
Contributor

Wow indeed this is a very elegant solution! Bravo!
I'll test it myself, and if everything runs fine I'll merge :)

Copy link
Contributor

@ElectreAAS ElectreAAS left a comment

Choose a reason for hiding this comment

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

LGTM!

@ElectreAAS ElectreAAS merged commit 0e6ea28 into ocurrent:main Apr 3, 2023
1 check passed
@punchagan punchagan deleted the separate-bench-repo branch April 3, 2023 12:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants