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(protoc-gen-pothos): fix object null check on toProto #438

Merged
merged 3 commits into from
Nov 29, 2024

Conversation

izumin5210
Copy link
Member

No description provided.

Copy link

changeset-bot bot commented Nov 29, 2024

🦋 Changeset detected

Latest commit: ab09c84

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
protoc-gen-pothos Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@izumin5210 izumin5210 changed the title fix(protoc-gen-pothos): fix objeet null check on toProto fix(protoc-gen-pothos): fix object null check on toProto Nov 29, 2024
@izumin5210 izumin5210 added the bug 🐛 Something isn't working label Nov 29, 2024
@izumin5210 izumin5210 merged commit bd49e47 into main Nov 29, 2024
8 of 10 checks passed
@izumin5210 izumin5210 deleted the izumin5210/fix-to-proto-func branch November 29, 2024 02:54
@github-actions github-actions bot mentioned this pull request Oct 1, 2024
izumin5210 pushed a commit that referenced this pull request Nov 29, 2024
This PR was opened by the [Changesets
release](https://github.com/changesets/action) 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
## @proto-graphql/codegen-core@0.5.2

### Patch Changes

- [#417](#417)
[`2ae48d1`](2ae48d1)
Thanks [@izumin5210](https://github.com/izumin5210)! - use
protobuf-es@v2 internally

- [#428](#428)
[`e3cee38`](e3cee38)
Thanks [@izumin5210](https://github.com/izumin5210)! - refactor: migrate
protoc plugins to ESM

## @proto-graphql/protoc-plugin-helpers@0.4.2

### Patch Changes

- [#417](#417)
[`2ae48d1`](2ae48d1)
Thanks [@izumin5210](https://github.com/izumin5210)! - use
protobuf-es@v2 internally

- [#427](#427)
[`c681582`](c681582)
Thanks [@izumin5210](https://github.com/izumin5210)! - refactor: parse
options with protoplugin api

- [#428](#428)
[`e3cee38`](e3cee38)
Thanks [@izumin5210](https://github.com/izumin5210)! - refactor: migrate
protoc plugins to ESM

- Updated dependencies
\[[`2ae48d1`](2ae48d1),
[`e3cee38`](e3cee38)]:
    -   @proto-graphql/codegen-core@0.5.2

## @proto-graphql/scalars-protobuf-es@0.4.2

### Patch Changes

- [#428](#428)
[`e3cee38`](e3cee38)
Thanks [@izumin5210](https://github.com/izumin5210)! - refactor: migrate
protoc plugins to ESM

## @proto-nexus/proto-fields-plugin@0.5.6

### Patch Changes

- [#417](#417)
[`2ae48d1`](2ae48d1)
Thanks [@izumin5210](https://github.com/izumin5210)! - use
protobuf-es@v2 internally

## protoc-gen-nexus@0.8.2

### Patch Changes

- [#417](#417)
[`2ae48d1`](2ae48d1)
Thanks [@izumin5210](https://github.com/izumin5210)! - use
protobuf-es@v2 internally

- [#427](#427)
[`c681582`](c681582)
Thanks [@izumin5210](https://github.com/izumin5210)! - refactor: parse
options with protoplugin api

- [#428](#428)
[`e3cee38`](e3cee38)
Thanks [@izumin5210](https://github.com/izumin5210)! - refactor: migrate
protoc plugins to ESM

- Updated dependencies
\[[`2ae48d1`](2ae48d1),
[`c681582`](c681582),
[`e3cee38`](e3cee38)]:
    -   @proto-graphql/protoc-plugin-helpers@0.4.2
    -   @proto-graphql/codegen-core@0.5.2

## protoc-gen-pothos@0.6.2

### Patch Changes

- [#417](#417)
[`2ae48d1`](2ae48d1)
Thanks [@izumin5210](https://github.com/izumin5210)! - use
protobuf-es@v2 internally

- [#438](#438)
[`bd49e47`](bd49e47)
Thanks [@izumin5210](https://github.com/izumin5210)! - fix object null
check on `toProto` func

- [#427](#427)
[`c681582`](c681582)
Thanks [@izumin5210](https://github.com/izumin5210)! - refactor: parse
options with protoplugin api

- [#428](#428)
[`e3cee38`](e3cee38)
Thanks [@izumin5210](https://github.com/izumin5210)! - refactor: migrate
protoc plugins to ESM

- Updated dependencies
\[[`2ae48d1`](2ae48d1),
[`c681582`](c681582),
[`e3cee38`](e3cee38)]:
    -   @proto-graphql/protoc-plugin-helpers@0.4.2
    -   @proto-graphql/codegen-core@0.5.2

Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug 🐛 Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant