Skip to content

[ci] Install ONNX before linting#4428

Merged
jcwchen merged 2 commits intoonnx:mainfrom
justinchuby:justinchu/build-onnx
Aug 10, 2022
Merged

[ci] Install ONNX before linting#4428
jcwchen merged 2 commits intoonnx:mainfrom
justinchuby:justinchu/build-onnx

Conversation

@justinchuby
Copy link
Copy Markdown
Member

Description

Install ONNX before linting

Motivation and Context

#4427

justinchuby and others added 2 commits August 10, 2022 17:17
@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
Copy Markdown
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 <justinchu@microsoft.com>

Signed-off-by: Justin Chu <justinchu@microsoft.com>
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