Skip to content

chore: introduce bin scripts including bin/run for running the tool - #361

Merged
G-Rath merged 8 commits into
mainfrom
create-run-script
Sep 2, 2026
Merged

chore: introduce bin scripts including bin/run for running the tool#361
G-Rath merged 8 commits into
mainfrom
create-run-script

Conversation

@G-Rath

@G-Rath G-Rath commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

This introduces some scripts to make it easier to run the tool, as well as help to keep things consistent.

To keep things simple I've opted to have bin/run just always build the image since that will only take a few seconds thanks to layer caching if nothing has changed, and that way you'll definitely be using the latest version of the tool

Resolves #340
Closes #336

@G-Rath
G-Rath force-pushed the create-run-script branch 2 times, most recently from 7d60500 to b38157e Compare September 1, 2026 22:47
@G-Rath
G-Rath requested a balanced review from Copilot September 1, 2026 22:53
@G-Rath
G-Rath marked this pull request as ready for review September 1, 2026 22:54

This comment was marked as outdated.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

🟢 Approval recommended

The scripts preserve existing container behavior while centralizing it without identified regressions.

Review details
  • Files reviewed: 1/4 changed files
  • Comments generated: 0 new
  • Review effort level: Balanced

@G-Rath
G-Rath merged commit 978d67a into main Sep 2, 2026
12 checks passed
@G-Rath
G-Rath deleted the create-run-script branch September 2, 2026 22:07
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.

Dockerfile: mount config, results, and base_urls OR document working docker run commands

4 participants