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

License issue #985

Open
1 task done
StefanTerdell opened this issue Aug 12, 2024 · 2 comments · Fixed by #986
Open
1 task done

License issue #985

StefanTerdell opened this issue Aug 12, 2024 · 2 comments · Fixed by #986
Labels
bug Something isn't working

Comments

@StefanTerdell
Copy link

StefanTerdell commented Aug 12, 2024

Confirm this is a Node library issue and not an underlying OpenAI API issue

  • This is an issue with the Node library

Describe the bug

Hi OpenAI! 👋🏻 It's really cool to see you're making good use of my package zod-to-json-schema! I see you've made some interesting changes in the vendored version. I'd be happy to take a look at them in the form of a PR and integrate them into the "canonical" version, which you can then use as a regular dependency.

In the meantime, to resolve the issue caused by the license, you can include the license and copyright notice in the vendored files

...or offer me a job 😉

To Reproduce

See ISC license conditions in https://github.com/StefanTerdell/zod-to-json-schema/blob/master/LICENSE

Code snippets

No response

OS

n/a

Node version

n/a

Library version

4.55.4

@StefanTerdell StefanTerdell added the bug Something isn't working label Aug 12, 2024
@RobertCraigie
Copy link
Collaborator

Hey @StefanTerdell, really sorry we messed up the licensing here!

you can include the license and copyright notice in the vendored files

Will add the LICENSE file 👍

To be clear, would you like us to add your license copy to the top of every vendored file? e.g.

/**
  ISC License

  Copyright (c) 2020, Stefan Terdell
  
  Permission to use, copy, modify, and/or distribute this software for any
  purpose with or without fee is hereby granted, provided that the above
  copyright notice and this permission notice appear in all copies.
  
  THE SOFTWARE IS PROVIDED "AS IS" AND THE AUTHOR DISCLAIMS ALL WARRANTIES
  WITH REGARD TO THIS SOFTWARE INCLUDING ALL IMPLIED WARRANTIES OF
  MERCHANTABILITY AND FITNESS. IN NO EVENT SHALL THE AUTHOR BE LIABLE FOR
  ANY SPECIAL, DIRECT, INDIRECT, OR CONSEQUENTIAL DAMAGES OR ANY DAMAGES
  WHATSOEVER RESULTING FROM LOSS OF USE, DATA OR PROFITS, WHETHER IN AN
  ACTION OF CONTRACT, NEGLIGENCE OR OTHER TORTIOUS ACTION, ARISING OUT OF
  OR IN CONNECTION WITH THE USE OR PERFORMANCE OF THIS SOFTWARE.
**/

const zodToJsonSchema = <Target extends Targets = 'jsonSchema7'>(
// ...

@StefanTerdell
Copy link
Author

Just including the original license file in the tree is fine. Do consider contributing to the original repo though!

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 a pull request may close this issue.

2 participants