Skip to content

[Harmony] Bug 2005381: Clean up docker configs, run tests locally easier #151

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
Open
justdave wants to merge 7 commits into
base: main
Choose a base branch
from
Open

[Harmony] Bug 2005381: Clean up docker configs, run tests locally easier #151

justdave wants to merge 7 commits into from

Conversation

@justdave
Copy link
Member

@justdave justdave commented Dec 11, 2025

Details

  • Cleaned some of the scripts related to docker builds, got the same "does docker exist?" sanity code in all of them now.
  • Converted the Release Tests GitHub Action to use a Docker image instead of running scripts directly on a GitHub test runner.
  • Moved the Release Tests action into the ci.yml file and removed releases.yml (getting all of the tests listed in the same place, since they all run on the same push/branch criteria anyway.
  • Added a script you can run with sh docker/run-tests-in-docker.sh which lets you pick a test to run and runs it locally.

Additional info

@justdave justdave requested a review from dylanwh December 11, 2025 06:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@dylanwh dylanwh Awaiting requested review from dylanwh

At least 1 approving review is required to merge this pull request.

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@justdave