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

Update linter and test dependencies to fix the failed workflow. #39

Merged
merged 2 commits into from
Apr 19, 2024

Conversation

LinZhihao-723
Copy link
Member

@LinZhihao-723 LinZhihao-723 commented Apr 18, 2024

References

Description

The current linter workflow failed because linter dependencies were outdated. One source file needs to be refactored after updating the linter versions. This PR also updates the test dependencies to lock smart_open version to 6.4.0 to make it compatible with Python3.6 and Python zstandard.

Validation performed

Ensured GH workflow passed.

@LinZhihao-723 LinZhihao-723 changed the title Update linter dependencies to fix the failed workflow. Update linter and test dependencies to fix the failed workflow. Apr 18, 2024
Copy link
Member

@davidlion davidlion left a comment

Choose a reason for hiding this comment

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

title fine for commit msg

@LinZhihao-723 LinZhihao-723 merged commit 3d69466 into y-scope:main Apr 19, 2024
8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants