Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

core: prevent creating a zero identifier with deserialization #797

Merged
merged 3 commits into from
Dec 10, 2024
Merged

core: prevent creating a zero identifier with deserialization #797

merged 3 commits into from
Dec 10, 2024

Conversation

conradoplg
Copy link
Contributor

Closes #793

@conradoplg
Copy link
Contributor Author

@Mergifyio update

@mergify Mergify
Copy link
Contributor

mergify bot commented Dec 9, 2024

update

✅ Branch has been successfully updated

mergify bot added a commit that referenced this pull request Dec 10, 2024
@mergify
Merge of #797
d58942c
@mergify mergify bot mentioned this pull request Dec 10, 2024
Closed
10 tasks
@mergify mergify bot merged commit d6c9f73 into main Dec 10, 2024
20 checks passed
@mergify mergify bot deleted the disallow-zero-identifier branch December 10, 2024 14:16
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers

@mpguerra mpguerra mpguerra approved these changes

@StackOverflowExcept1on StackOverflowExcept1on StackOverflowExcept1on approved these changes

@natalieesk natalieesk Awaiting requested review from natalieesk

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Identifier allows zero scalar
3 participants
@conradoplg @mpguerra @StackOverflowExcept1on