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

chore: update black version in tox #714

Open
wants to merge 11 commits into
base: master
Choose a base branch
from
Open

Conversation

tushar-composio
Copy link

@tushar-composio tushar-composio commented Oct 16, 2024

Black has had several bugfixes and style tweaks the past 1.5 years, this brings our codebase up to date.

Added flake8 ignore of rule E704 is for lines like these:

@t.overload
def foo() -> int: ...

E704 complains about mulitple statements on the same line. It's not relevant to us as black will reformat any bad cases anyway.

@tushar-composio tushar-composio marked this pull request as ready for review October 16, 2024 11:27
Copy link

github-actions bot commented Oct 16, 2024

This comment was generated by github-actions[bot]!

JS SDK Coverage Report

📊 Coverage report for JS SDK can be found at the following URL:
https://pub-92e668239ab84bfd80ee07d61e9d2f40.r2.dev/coverage-11365904066/coverage/index.html

📁 Test report folder can be found at the following URL:
https://pub-92e668239ab84bfd80ee07d61e9d2f40.r2.dev/html-report-11365904066/html-report/report.html

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.

1 participant