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

Remove the implementations folder since it is not maintained #87

Merged
merged 2 commits into from
Feb 3, 2024

Conversation

smarr
Copy link
Owner

@smarr smarr commented Feb 3, 2024

The main goal of this PR is to clean up the repository, and reduce the number of things floating around that are not actually used.

The new convention is to have a build.sh script per language.
The script, if necessary builds the benchmarks.

build.sh ${langId} should be supported if there are slight difference between concrete language implementations, that need a build step.

build.sh style should run language-specific style checkers on the benchmark code.

- move all build instructions to a build.sh in the language-specific folder
- update GitHub Actions to avoid deprecation warnings
- remove implementations folder
  - it contained various unmaintained and untested scripts
- simplify rebench.conf
  - it’s not tested, so, this is more of a template of how a setup may look like

Signed-off-by: Stefan Marr <git@stefan-marr.de>
Signed-off-by: Stefan Marr <git@stefan-marr.de>
@smarr smarr merged commit 329686c into master Feb 3, 2024
18 checks passed
@smarr smarr deleted the remove-impl branch February 3, 2024 17:02
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.

1 participant