Skip to content

Conversation

@garymm
Copy link
Contributor

@garymm garymm commented Mar 21, 2022

Microsoft doesn't author this package, the ONNX community does.

Copied from onnx/onnx:
https://github.com/onnx/onnx/blob/acc127219b45bc27b0180b1fdc08299eac81b167/setup.py#L342

@hwangdeyu
Copy link
Contributor

It should be signed off for each of commit in tf2onnx.

Copy link
Contributor

@hwangdeyu hwangdeyu left a comment

Choose a reason for hiding this comment

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

LGTM. Please fix CI DCO required. Thanks!

garymm added 2 commits March 23, 2022 22:21
Microsoft doesn't author this package, the ONNX community does. Copied from onnx/onnx: https://github.com/onnx/onnx/blob/acc127219b45bc27b0180b1fdc08299eac81b167/setup.py#L342 Signed-off-by: Gary Miguel <garymiguel@microsoft.com>
Signed-off-by: Gary Miguel <garymiguel@microsoft.com>
@garymm
Copy link
Contributor Author

garymm commented Mar 23, 2022

DCO check passed. Please merge. Thanks!

@hwangdeyu hwangdeyu merged commit c83a6f8 into onnx:main Mar 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

3 participants