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

Version Packages #588

Merged
merged 1 commit into from
Aug 4, 2024
Merged

Version Packages #588

merged 1 commit into from
Aug 4, 2024

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Aug 3, 2024

This PR was opened by the Changesets release GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated.

Releases

@preact/signals-core@1.8.0

Minor Changes

  • #587 cd9efbb Thanks @JoviDeCroock! - Adjust the ReadOnlySignal type to not inherit from Signal
    this way the type can't be widened without noticing, i.e. when
    we'd have

    const sig: Signal = useComputed(() => x);

    We would have widened the type to be mutable again, which for
    a computed is not allowed. We want to provide the tools to our
    users to avoid these footguns hence we are correcting this type
    in a minor version.

Copy link

netlify bot commented Aug 3, 2024

Deploy Preview for preact-signals-demo ready!

Name Link
🔨 Latest commit 4b2be49
🔍 Latest deploy log https://app.netlify.com/sites/preact-signals-demo/deploys/66ae2069bcf011000891a523
😎 Deploy Preview https://deploy-preview-588--preact-signals-demo.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@JoviDeCroock JoviDeCroock merged commit 8fab8fe into main Aug 4, 2024
4 checks passed
@JoviDeCroock JoviDeCroock deleted the changeset-release/main branch August 4, 2024 06:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant