Skip to content

Support running unit tests when files are saved - #87

Merged
dnephin merged 1 commit into
docker:masterfrom
dnephin:add-watcher
May 17, 2017
Merged

dnephin merged 1 commit into
docker:masterfrom
dnephin:add-watcher

Conversation

@dnephin

@dnephin dnephin commented May 15, 2017

Copy link
Copy Markdown
Contributor

This is a workflow that I've used for a long time. I think it works very well, and I'd like to include it as part of the build tools so that others can use it.

Here's how it works:

Start the watcher in one terminal

make -f docker.Makefile watch

Make code changes in another terminal, editor, or IDE. Every time you save a file the unit tests for that package will run automatically.

Get (almost) instantaneous feedback on your code changes!

Signed-off-by: Daniel Nephin <dnephin@docker.com>

@vdemeester vdemeester left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM 👏🐳
Using goconvey right now but it is runs all packages each time 😅 so this one is really good !

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

LGTM

@dnephin
dnephin merged commit 3d655c0 into docker:master May 17, 2017
@GordonTheTurtle GordonTheTurtle added this to the 17.06.0 milestone May 17, 2017
@dnephin
dnephin deleted the add-watcher branch May 17, 2017 14:19
trapier pushed a commit to trapier/cli that referenced this pull request Sep 30, 2019
…ildtime

[master] [TAR-167] Fix windows version output
nobiit pushed a commit to nobidev/docker-cli that referenced this pull request Nov 19, 2025
Do not call the config endpoint if API is lower than 1.30
nobiit pushed a commit to nobidev/docker-cli that referenced this pull request Nov 19, 2025
Remove ubuntu-zesty from the default DOCKER_BUILD_PKGS in Makefile
Upstream-commit: 2ab6d8ab3a9da8917ef52b87943e3de83a1092a5
Component: packaging
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants