nuxt-modules / turnstile

🔥 Cloudflare Turnstile integration for Nuxt
https://cloudflare.com/products/turnstile
MIT License
215 stars 17 forks source link

chore: release v0.5.1 #265

Closed dargmuesli closed 10 months ago

dargmuesli commented 10 months ago

Proposed release for type fix included in https://github.com/nuxt-modules/turnstile/pull/263

danielroe commented 10 months ago

I don't believe any release is required as the only change was to this module's devDependencies:

https://github.com/nuxt-modules/turnstile/compare/v0.5.0...main

codecov-commenter commented 10 months ago

Codecov Report

Merging #265 (87796b6) into main (2cd1512) will not change coverage. The diff coverage is n/a.

:exclamation: Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

@@           Coverage Diff           @@
##             main     #265   +/-   ##
=======================================
  Coverage   82.60%   82.60%           
=======================================
  Files           2        2           
  Lines         115      115           
  Branches       13       13           
=======================================
  Hits           95       95           
  Misses         20       20           

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

danielroe commented 10 months ago

Ahh - you mean you're getting a runtime error from type checking the Turnstile component?

dargmuesli commented 10 months ago

Yes, this is the fix I think to release: https://github.com/nuxt-modules/turnstile/compare/v0.5.0...main#diff-bc1c2b688c03023c444393cabce8b293da4e7cf5cdba97ef256748cdee6183f9R36

dargmuesli commented 10 months ago

Hmm, the link doesn't really point to the right tab when clicking on it. I mean the component file as you mentioned and the included type fix, which I squashed into the dev dependency update, so it doesn't show up as separate commit anymore.

dargmuesli commented 10 months ago

So it's not really a runtime error first for me, but a linting error downstream (https://github.com/maevsi/maevsi/pull/1347) that I think can only be resolved with an update to this package (and thus to the included component).

danielroe commented 10 months ago

Understood. I'll release a patch.