Skip to content

Conversation

@Pavel-Durov
Copy link
Contributor

Add -f/--file CLI flag to load alternative haste configuration files. Defaults to haste.toml when not specified. This is useful when running multiple benchmarks with different configurations.

Add -f/--file CLI flag to load alternative haste configuration
files. Defaults to haste.toml when not specified. This is useful when
running multiple benchmarks with different configurations.
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds support for specifying custom configuration files via a -f/--file CLI option, enabling users to run benchmarks with different configurations without modifying the default haste.toml file.

Key Changes:

  • Added --file CLI flag to specify custom config file paths
  • Updated App struct to store and use the config file path
  • Added test coverage for default config path and config file loading

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@vext01
Copy link
Contributor

vext01 commented Oct 20, 2025

I have an almost identical diff that I've been using locally :)

The only comment I have is CONFIG_FILE should probably be renamed DEFAULT_CONFIG_FILE or similar.

@Pavel-Durov
Copy link
Contributor Author

I have an almost identical diff that I've been using locally :)

The only comment I have is CONFIG_FILE should probably be renamed DEFAULT_CONFIG_FILE or similar.

Renamed 👉 af48516

@vext01 vext01 added this pull request to the merge queue Oct 20, 2025
Merged via the queue into ykjit:main with commit 6016872 Oct 20, 2025
2 checks passed
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.

2 participants