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

[ci] Install ONNX before linting #4428

Merged
merged 2 commits into from
Aug 10, 2022
Merged

Conversation

justinchuby
Copy link
Contributor

Description

Install ONNX before linting

Motivation and Context

#4427

@justinchuby justinchuby marked this pull request as ready for review August 10, 2022 17:19
@justinchuby justinchuby requested a review from a team as a code owner August 10, 2022 17:19
Copy link
Member

@jcwchen jcwchen left a comment

Choose a reason for hiding this comment

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

LGTM. Let's forward this first and see whether it resolves the pylint issue in #4427.

@jcwchen jcwchen enabled auto-merge (squash) August 10, 2022 17:43
@jcwchen jcwchen added the module: CI pipelines Issues related to the CI pipeline label Aug 10, 2022
@jcwchen jcwchen merged commit ff95f7d into onnx:main Aug 10, 2022
broune pushed a commit to broune/onnx that referenced this pull request May 6, 2023
Signed-off-by: Justin Chu <[email protected]>

Signed-off-by: Justin Chu <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
module: CI pipelines Issues related to the CI pipeline
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants