Skip to content

Commit

Permalink
Fix remaining issues
Browse files Browse the repository at this point in the history
Signed-off-by: Keith Battocchi <[email protected]>
  • Loading branch information
kbattocchi committed Aug 11, 2024
1 parent f2ec54e commit f7c63ac
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 6 deletions.
1 change: 0 additions & 1 deletion econml/tests/test_automated_ml.py
Original file line number Diff line number Diff line change
Expand Up @@ -28,7 +28,6 @@
import os

try:
from azureml.train.automl.exceptions import ClientException
from azureml.core.authentication import AzureCliAuthentication
from econml.automated_ml import addAutomatedML, setAutomatedMLWorkspace, EconAutoMLConfig
AutomatedTLearner = addAutomatedML(TLearner)
Expand Down
6 changes: 1 addition & 5 deletions pyproject.toml
Original file line number Diff line number Diff line change
Expand Up @@ -144,10 +144,6 @@ quote-style = "preserve"

[tool.ruff.lint]
ignore = [
"F401", # Unused import
"F541", # f-string is missing placeholders
"F811", # Redefinition of unused name from line N
"E713", # Test for membership should be 'not in'
"D100", # Missing docstring in public module
"D101", # Missing docstring in public class
"D102", # Missing docstring in public method
Expand All @@ -159,7 +155,7 @@ ignore = [
extend-select = [
"D", # Docstring
]
extend-per-file-ignores = { "econml/tests" = ["D"] }
extend-per-file-ignores = { "econml/tests" = ["D"] } # ignore docstring rules for tests

[tool.ruff.lint.pydocstyle]
convention = "numpy"

0 comments on commit f7c63ac

Please sign in to comment.