Skip to content
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

[now dev] Add warning when there are no build matches #2594

Merged
merged 2 commits into from Jul 23, 2019

Conversation

TooTallNate
Copy link
Member

This matches the error in production, except it's just a non-fatal warning when running in now dev. This is so that the user can fix the warning without having to restart the now dev server.

This matches the error in production, except it's just a non-fatal
warning when running in `now dev`. This is so that the user can fix
the warning without having to restart the `now dev` server.
Copy link
Contributor

@AndyBitz AndyBitz left a comment

Choose a reason for hiding this comment

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

👍

@codecov-io
Copy link

codecov-io commented Jul 23, 2019

Codecov Report

Merging #2594 into canary will decrease coverage by 0.19%.
The diff coverage is 66.66%.

Impacted file tree graph

@@           Coverage Diff            @@
##           canary   #2594     +/-   ##
========================================
- Coverage    8.17%   7.97%   -0.2%     
========================================
  Files         257     257             
  Lines        9436    9376     -60     
  Branches     1089    1062     -27     
========================================
- Hits          771     748     -23     
+ Misses       8612    8579     -33     
+ Partials       53      49      -4
Impacted Files Coverage Δ
src/util/dev/builder.ts 12.57% <66.66%> (+1.12%) ⬆️
src/util/dev/builder-cache.ts 29.54% <0%> (-2.3%) ⬇️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a29890c...f55c865. Read the comment docs.

@TooTallNate TooTallNate merged commit c413cc7 into canary Jul 23, 2019
@TooTallNate TooTallNate deleted the update/now-dev-no-builds-warning branch July 23, 2019 18:12
AndyBitz pushed a commit that referenced this pull request Jul 25, 2019
* [now dev] Add warning when there are no build matches

This matches the error in production, except it's just a non-fatal
warning when running in `now dev`. This is so that the user can fix
the warning without having to restart the `now dev` server.

* Add integration test
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.

None yet

4 participants