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

Remove some non null assertion operator #1057

Merged

Conversation

kody-liu
Copy link
Contributor

@kody-liu kody-liu commented Sep 7, 2024

I removed some non-null assertion operators, which should fix this issue: #968.

By the way, does anyone know how to install BlockNote via a GitHub repository URL? I tried adding a prepare script but encountered an error. You can see more details in my comment below.

Copy link

vercel bot commented Sep 7, 2024

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Updated (UTC)
blocknote ✅ Ready (Inspect) Visit Preview Sep 18, 2024 5:08pm
blocknote-website ✅ Ready (Inspect) Visit Preview Sep 18, 2024 5:08pm

Copy link

vercel bot commented Sep 7, 2024

@kody-liou is attempting to deploy a commit to the TypeCell Team on Vercel.

A member of the Team first needs to authorize it.

package.json Outdated Show resolved Hide resolved
Copy link
Collaborator

@matthewlipski matthewlipski left a comment

Choose a reason for hiding this comment

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

The non-null assertion changes look good, but what's up with the version bump? I assume it's smth to do with the prepare script so do you could just keep the non-null fixes in this PR and create a new issue/PR for the script?

@kody-liu
Copy link
Contributor Author

Done. I’ve removed the prepare script and the version bump. This PR now only includes the non-null assertion changes.

@matthewlipski
Copy link
Collaborator

Awesome thx!

@matthewlipski matthewlipski self-requested a review September 12, 2024 14:33
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.

2 participants