Skip to content

Bug 2052751: run tests from docker FASTER#167

Merged
justdave merged 4 commits into
mainfrom
bug-2052751-harmony
Jul 6, 2026
Merged

Bug 2052751: run tests from docker FASTER#167
justdave merged 4 commits into
mainfrom
bug-2052751-harmony

Conversation

@justdave

@justdave justdave commented Jul 6, 2026

Copy link
Copy Markdown
Member

Details

Only copy enough to run the dependency install before installing dependencies, so that you don't have to reinstall dependencies any time you touch anything in the repository.

Additional info

Test Plan

  1. sh docker/run-tests-in-docker.sh - pick option 6
  2. make a material change to any file other than Bugzilla.pm, Makefile.PL, or gen-cpanfile.sh
  3. Do step 1 again, notice it starts running tests within 10 seconds. instead of waiting 5 to 10 minutes for dependency install.

@justdave justdave requested review from dylanwh and mrenvoize July 6, 2026 03:31
@justdave

justdave commented Jul 6, 2026

Copy link
Copy Markdown
Member Author

The last two commits basically make it so you can run prove and perl -c from the command line in your development environment via the docker image without needing dependencies installed on the host. It was close enough related to this PR I figured I'd add it on here instead of filing a new bug.

@mrenvoize mrenvoize 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.

Nice clear improvements, works in testing, approving.

@justdave justdave merged commit 1816a4b into main Jul 6, 2026
8 checks passed
@justdave justdave deleted the bug-2052751-harmony branch July 6, 2026 07:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants