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

fix(nuxt-start): add missing vue-client-only dependency #6267

Merged
merged 1 commit into from
Aug 21, 2019

Conversation

pimlie
Copy link

@pimlie pimlie commented Aug 21, 2019

Resolves: #6266

Types of changes

  • Bug fix (a non-breaking change which fixes an issue)
  • New feature (a non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Description

Checklist:

  • My change requires a change to the documentation.
  • I have updated the documentation accordingly. (PR: #)
  • I have added tests to cover my changes (if not applicable, please state why)
  • All new and existing tests are passing.

@pimlie
Copy link
Author

pimlie commented Aug 21, 2019

Dont know why Circle CI has issues, already restarted the job twice but that didnt resolve it.

@pimlie pimlie requested a review from a team August 21, 2019 15:32
@codecov-io
Copy link

Codecov Report

Merging #6267 into dev will increase coverage by 0.03%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##              dev    #6267      +/-   ##
==========================================
+ Coverage   95.72%   95.76%   +0.03%     
==========================================
  Files          80       80              
  Lines        2666     2666              
  Branches      687      687              
==========================================
+ Hits         2552     2553       +1     
+ Misses         98       97       -1     
  Partials       16       16
Flag Coverage Δ
#e2e 100% <ø> (ø) ⬆️
#fixtures 50.63% <ø> (ø) ⬆️
#unit 92.46% <ø> (+0.03%) ⬆️
Impacted Files Coverage Δ
packages/vue-renderer/src/renderer.js 94.3% <0%> (+0.81%) ⬆️

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 662a409...27f39bc. Read the comment docs.

@pi0 pi0 merged commit 19f7fb1 into nuxt:dev Aug 21, 2019
@pi0 pi0 mentioned this pull request Aug 22, 2019
@pimlie pimlie deleted the fix-nuxt-start-dependency branch August 25, 2019 16:11
atinux pushed a commit that referenced this pull request Aug 28, 2019
* [release]

* fix(vue-app): enforce default css when used with frameworks (#6255)

* chore(deps): update dependency serialize-javascript to ^1.8.0 (#6254)

* fix(config): provide typescript backward compatibility (#6258)

* fix(vue-app): only overwrite leave listener when none provided or without done arg (#6264)

* fix(nuxt-start): add missing `vue-client-only` dependency (#6267)

* fix(server): treat `https: null` as `https: undefined` (#6265)

* chore(deps): update devdependency rollup to ^1.20.0 (#6268)

* fix: improve watching experience for generated files (#6257)

* refactor(core): use hable (#6271)

* test: disable terser/minify by default (#6290)

* chore(deps): update devdependency rollup to ^1.20.1 (#6282)

* chore(cli): accept hooks (#6274)

* chore(deps): update devdependency rollup-plugin-alias to v2 (#6281)

* fix(builder): apply overrides from app dir only (#6283)

[release]

* feat: function watchQuery (#6245)

* Revert "feat: function watchQuery (#6245)" (#6296)

This reverts commit 3c61830.

* chore(deps): update devdependency rollup to ^1.20.2 (#6295)



Co-authored-by: Renovate Bot <bot@renovateapp.com>

* test(vue-app): add template compiler helper (#6299)

* test: add jest roots for less greedy test search (#6300)

* test: remove unnecessary generate (#6301)

* fix: update the minimal require node version in distributions (#6310)

* chore(deps): update devdependency babel-eslint to ^10.0.3 (#6304)

* chore(deps): lock file maintenance (#6305)

* chore: upgrade circleci config to v2.1 (#6312)

* test: add test for modern bundle size (#6302)

* test: add client-only test to basic fixture (#6315)

test: check for no-ssr deprecation warning
@danielroe danielroe added the 2.x label Jan 18, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Can't anymore use nuxt as devDependencies, because nuxt-start is missing vue-client-only
4 participants