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

style: use ruff #313

Merged
merged 13 commits into from
Mar 29, 2024
Merged

style: use ruff #313

merged 13 commits into from
Mar 29, 2024

Conversation

jsstevenson
Copy link
Member

No description provided.

@jsstevenson jsstevenson added the priority:low Low priority label Mar 29, 2024

[tool.ruff]
src = ["src"]
exclude = ["docs/*", "analysis/*", "codebuild/*"]
Copy link
Member Author

Choose a reason for hiding this comment

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

I went ahead and ran format on analysis and codebuild but I didn't think it would be worth it to actively maintain them style-wise

@jsstevenson jsstevenson marked this pull request as ready for review March 29, 2024 19:03
@jsstevenson jsstevenson requested a review from korikuzma March 29, 2024 19:07
@@ -25,8 +25,8 @@ requires-python = ">=3.8"
description = "A search interface for cancer variant interpretations assembled by aggregating and harmonizing across multiple cancer variant interpretation knowledgebases."
license = {file = "LICENSE"}
dependencies = [
"ga4gh.vrs==~=2.0.0a5",
"gene-normalizer[etl]==~=0.3.0-dev1",
Copy link
Member

Choose a reason for hiding this comment

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

ope

Copy link
Member Author

Choose a reason for hiding this comment

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

im growing and learning

Copy link
Member

@korikuzma korikuzma left a comment

Choose a reason for hiding this comment

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

🧽

@jsstevenson jsstevenson merged commit 0c7b328 into epic-231 Mar 29, 2024
13 checks passed
@jsstevenson jsstevenson deleted the ruff branch March 29, 2024 19:14
@korikuzma korikuzma mentioned this pull request Apr 1, 2024
4 tasks
@korikuzma korikuzma mentioned this pull request May 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority:low Low priority
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants