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 ci #116

Merged
merged 1 commit into from
Nov 16, 2022
Merged

fix ci #116

merged 1 commit into from
Nov 16, 2022

Conversation

odashi
Copy link
Collaborator

@odashi odashi commented Nov 16, 2022

Overview

Fix CI script.

Details

pflake8 couldn't propagate errno if it is invoked as a module. This change fixes it and some flake8 errors in main.
Additionally, I removed some unnecessary installation in the black and flake8 tests.

References

Blocked by

NA

@odashi odashi requested a review from ShigekiKarita November 16, 2022 08:06
@odashi
Copy link
Collaborator Author

odashi commented Nov 16, 2022

Since I need to merge this ASAP to proceed other PR, I will skip reviewing for this particular PR. I confirmed that all CIs are working as expected by checking the log manually.

@odashi odashi merged commit 3c94e67 into main Nov 16, 2022
@odashi odashi deleted the fix-ci-2 branch November 16, 2022 08:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant