-
Notifications
You must be signed in to change notification settings - Fork 220
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.
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
fix: disable webpack
rebuilds on karma --single-run
#259
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we add test for this to avoid regressions?
There aren't any actual tests in this repo beyond the basic integration one. The next weekend I have free I can sit down and write some in a follow up PR 😄 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good, let's add tests in future 👍
d867585
to
7f0c33c
Compare
webpack
rebuilds on karma --single-run
merge? travis red. Almost 1 month after approved. |
527cfdc
to
a8cff06
Compare
a8cff06
to
2074f60
Compare
Please check if the PR fulfills these requirements
What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)
What is the current behavior? (You can also link to an open issue here)
When running a long test suite as a single run, if you make changes to any of the files during the run, webpack will start recompiling
What is the new behavior?
Webpack is only run in watch mode when running karma is not running with the
--single-run
flagDoes this PR introduce a breaking change?
If this PR contains a breaking change, please describe the following...
N/A
Other information: