]> git.ipfire.org Git - thirdparty/vuejs/pinia.git/commitdiff
chore: improve message
authorEduardo San Martin Morote <posva13@gmail.com>
Thu, 12 Aug 2021 16:55:31 +0000 (18:55 +0200)
committerEduardo San Martin Morote <posva@users.noreply.github.com>
Fri, 13 Aug 2021 08:14:53 +0000 (10:14 +0200)
.github/PULL_REQUEST_TEMPLATE.md
src/store.ts

index a7761bd8a6e2aa7fa036db9389820052193a3171..4ebd805a7883b52bb2831792c00b109824e7515a 100644 (file)
@@ -1,36 +1,4 @@
 <!--
-Please make sure to read the Pull Request Guidelines:
-https://github.com/vuejs/vue/blob/dev/.github/CONTRIBUTING.md#Pull-Request
+Please make sure to include a test! If this is closing an
+existing issue, reference that issue as well.
 -->
-
-<!-- PULL REQUEST TEMPLATE -->
-<!-- (Update "[ ]" to "[x]" to check a box) -->
-<!-- Tip: publish the PR and check the checkboxes by simply clicking on them -->
-
-**What kind of change does this PR introduce?** (check at least one)
-
-- [ ] Bugfix
-- [ ] Feature
-- [ ] Code style update
-- [ ] Refactor
-- [ ] Build-related changes
-- [ ] Other, please describe:
-
-**Does this PR introduce a breaking change?** (check one)
-
-- [ ] Yes
-- [ ] No
-
-If yes, please describe the impact and migration path for existing applications:
-
-**The PR fulfills these requirements:**
-
-- [ ] When resolving a specific issue, it's referenced in the PR's title (e.g. `fix #xxx[,#xxx]`, where "xxx" is the issue number)
-- [ ] All tests are passing
-- [ ] New/updated tests are included
-
-If adding a **new feature**, the PR's description includes:
-
-- [ ] A convincing reason for adding this feature (to avoid wasting your time, it's best to open a suggestion issue first and wait for approval before working on it)
-
-**Other information:**
index 691ca1e813ed13626f90aa9786343b5d3ead14c0..684c3027031ef127d28709bf5a1ca5954fbf81cd 100644 (file)
@@ -409,15 +409,17 @@ function createSetupStore<
     $patch,
     $reset,
     $subscribe(callback, options = {}) {
+      /* istanbul ignore if */
       if (__DEV__ && typeof options === 'boolean') {
         console.warn(
           `[🍍]: store.$subscribe() no longer accepts a boolean as the 2nd parameter:\n` +
             `Replace "store.$subscribe(fn, ${String(
               options
-            )})" with "$store.$subscribe(fn, { detached: ${String(
+            )})" with "store.$subscribe(fn, { detached: ${String(
               options
-            )} })".`
+            )} })".\n This will fail in production.`
         )
+        options = { detached: options }
       }
 
       const _removeSubscription = addSubscription(