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.

Sign up for GitHub

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

Jump to bottom

useChangeCase: Breaking change on change-case #3457

Closed
7 tasks done
LynxTR opened this issue Oct 5, 2023 · 1 comment · Fixed by #3460 · 4 remaining pull requests
Closed
7 tasks done

useChangeCase: Breaking change on change-case #3457

LynxTR opened this issue Oct 5, 2023 · 1 comment · Fixed by #3460 · 4 remaining pull requests

Comments

@LynxTR
Copy link

LynxTR commented Oct 5, 2023

Describe the bug

With the release of change-case@5, there were breaking changes in header case and param case: https://github.com/blakeembrey/change-case/releases/tag/change-case%405.0.0

✘ [ERROR] No matching export in "node_modules/.pnpm/[email protected]/node_modules/change-case/dist/index.js" for import "headerCase"

    node_modules/.pnpm/@[email protected][email protected][email protected]/node_modules/@vueuse/integrations/useChangeCase.mjs:3:56:
      3 │ import { camelCase, capitalCase, constantCase, dotCase, headerCase, noCase, paramCase, pascalCase, pathCase, sentenceCase, snakeCase } from 'change-case';
        ╵                                                         ~~~~~~~~~~

✘ [ERROR] No matching export in "node_modules/.pnpm/[email protected]/node_modules/change-case/dist/index.js" for import "paramCase"

    node_modules/.pnpm/@[email protected][email protected][email protected]/node_modules/@vueuse/integrations/useChangeCase.mjs:3:76:
      3 │ import { camelCase, capitalCase, constantCase, dotCase, headerCase, noCase, paramCase, pascalCase, pathCase, sentenceCase, snakeCase } from 'change-case';
        ╵                                                                             ~~~~~~~~~

Reproduction

https://github.com/blakeembrey/change-case

System Info

System:
    OS: Linux 6.2 Ubuntu 23.04 23.04 (Lunar Lobster)
    CPU: (16) x64 AMD Ryzen 7 5700U with Radeon Graphics
    Memory: 5.88 GB / 13.50 GB
    Container: Yes
    Shell: 5.2.15 - /bin/bash
  Binaries:
    Node: 18.18.0 - ~/.nvm/versions/node/v18.18.0/bin/node
    npm: 9.8.1 - ~/.nvm/versions/node/v18.18.0/bin/npm
    pnpm: 8.8.0 - ~/.nvm/versions/node/v18.18.0/bin/pnpm
  Browsers:
    Chrome: 117.0.5938.88
  npmPackages:
    @vueuse/core: ^10.4.1 => 10.4.1 
    @vueuse/integrations: ^10.4.1 => 10.4.1 
    vue: ^3.2.30 => 3.3.4

Used Package Manager

pnpm

Validations
wheatjs added a commit that referenced this issue Oct 6, 2023
@wheatjs
fix: closes #3457
0473413
@wheatjs wheatjs mentioned this issue Oct 6, 2023
Merged
5 tasks
@stale Stale
Copy link

stale bot commented Dec 4, 2023

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Dec 4, 2023
@stale stale bot closed this as completed Dec 11, 2023
github-merge-queue bot pushed a commit that referenced this issue Jun 12, 2024
@wheatjs @antfu
feat(useChangeCase)!: migrate to change-case@v5, closes #3457 (#3460)
df343af 
Co-authored-by: Anthony Fu <[email protected]>
@renovate renovate bot mentioned this issue Aug 20, 2024
Open
1 task
This was referenced Sep 7, 2024
Open
Open
Open
Open
Open
Open
Open
@snyk-io snyk-io bot mentioned this issue Sep 13, 2024
Open
@gn00029914 gn00029914 mentioned this issue Sep 15, 2024
Open
@snyk-io snyk-io bot mentioned this issue Sep 15, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feat(useChangeCase)!: migrate to change-case@v5, closes #3457 vueuse/vueuse
[Snyk] Upgrade @vueuse/core from 10.11.1 to 11.0.0 gn00029914/vite-app-demo
[Snyk] Upgrade @vueuse/components from 10.11.1 to 11.0.0 gn00029914/vite-app-demo
[Snyk] Upgrade @vueuse/integrations from 10.11.1 to 11.0.0 gn00029914/vite-app-demo
[Snyk] Upgrade: , , gn00029914/vite-app-demo
[Snyk] Upgrade @vueuse/core from 10.11.1 to 11.0.1 gn00029914/vite-app-demo
[Snyk] Upgrade @vueuse/integrations from 10.11.1 to 11.0.1 gn00029914/vite-app-demo
[Snyk] Upgrade @vueuse/components from 10.11.1 to 11.0.1 gn00029914/vite-app-demo
[Snyk] Upgrade: , , , , axios-mock-adapter, pinia, postcss-focus-visible Organic-Hemp-mexico/openverse
[Snyk] Upgrade: , , gn00029914/vite-app-demo
1 participant
@LynxTR