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: resolve type conflict by updating primevue #61

Merged

Conversation

BobbieGoede
Copy link
Contributor

This updates primevue to the latest version which includes primefaces/primevue#5419 to prevent nuxt/nuxt#26214 from happening.

@cagataycivici
Copy link
Member

Thanks, cc @mertsincan

@@ -44,7 +44,7 @@
},
"dependencies": {
"@nuxt/kit": "^3.7.3",
"primevue": "~3.49.1"
"primevue": "~3.51.0"
Copy link

@DamianGlowala DamianGlowala Apr 9, 2024

Choose a reason for hiding this comment

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

Current approach of allowing patch-only updates might be too restrictive and relies on maintainers to act, potentially locking end users from getting the latest updates as soon as they get released (including myself).

Suggested change
"primevue": "~3.51.0"
"primevue": "^3.51.0"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I agree this would be the better approach but wasn't sure if there is a specific reasoning behind version pinning so I kept that as is, I'll leave it up to the maintainers whether this should be more permissive.

/cc @mertsincan

@martinszeltins
Copy link

It would be great to get this merged, otherwise types are broken for a Nuxt project without manual resolutions workaround.

@mertsincan

@mertsincan mertsincan added this to the 0.3.1 milestone May 24, 2024
@mertsincan mertsincan added the bug Something isn't working label May 24, 2024
@mertsincan
Copy link
Member

Thanks a lot for your contribution!

@mertsincan mertsincan merged commit 8d4872b into primefaces:main May 24, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants