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: initialize undefined optional fields upon use #802

Merged
merged 2 commits into from
Mar 19, 2023
Merged

fix: initialize undefined optional fields upon use #802

merged 2 commits into from
Mar 19, 2023

Conversation

davidzeng0
Copy link
Contributor

No description provided.

@stephenh
Copy link
Owner

@davidzeng0 I've merged your other two PRs, so this one has a conflict now; could you rebase it? Thanks!

@davidzeng0
Copy link
Contributor Author

Rebased

@stephenh
Copy link
Owner

@davidzeng0 Ah great, thanks for rebasing; can you try and find an existing integration/* test that matches the parameters.txt you're using and add a reproduction of what would previously blow up, but now does not?

Just to keep this behavior from regressing.

Also, do you think this PR will fix this issue:

#788

My guess is yes?

Thanks!

@davidzeng0
Copy link
Contributor Author

This should fix #788 and #736

@davidzeng0
Copy link
Contributor Author

As expected, the test fails without 6f6f97d

@stephenh
Copy link
Owner

Great integration test, thank you!

@stephenh stephenh merged commit ee52e06 into stephenh:main Mar 19, 2023
@davidzeng0 davidzeng0 deleted the initialize branch March 19, 2023 19:18
stephenh pushed a commit that referenced this pull request Mar 19, 2023
# [1.143.0](v1.142.1...v1.143.0) (2023-03-19)

### Bug Fixes

* initialize undefined optional fields upon use ([#802](#802)) ([ee52e06](ee52e06))

### Features

* group encoding and decoding support ([#799](#799)) ([5ebe3c0](5ebe3c0))

### Performance Improvements

* use array.push to prevent reallocation on every field ([#804](#804)) ([a6aea2c](a6aea2c))
@stephenh
Copy link
Owner

🎉 This PR is included in version 1.143.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants