From ee90017d3f9845ca7a2fc482106744213f8dcfd8 Mon Sep 17 00:00:00 2001 From: Dhruv Manilawala Date: Wed, 3 Jul 2024 13:55:29 +0530 Subject: [PATCH] Remove `demisto/content` from ecosystem checks (#12160) ## Summary Follow-up to https://github.com/astral-sh/ruff/pull/12129 to remove the `demisto/content` from ecosystem checks. The previous PR removed it from the deprecated script which I didn't notice until recently. ## Test Plan Ecosystem comment --- .../ruff-ecosystem/ruff_ecosystem/defaults.py | 32 ++++++++++++------- 1 file changed, 21 insertions(+), 11 deletions(-) diff --git a/python/ruff-ecosystem/ruff_ecosystem/defaults.py b/python/ruff-ecosystem/ruff_ecosystem/defaults.py index 95a61b05e8..9350513227 100644 --- a/python/ruff-ecosystem/ruff_ecosystem/defaults.py +++ b/python/ruff-ecosystem/ruff_ecosystem/defaults.py @@ -4,7 +4,6 @@ Default projects for ecosystem checks from ruff_ecosystem.projects import ( CheckOptions, - FormatOptions, Project, Repository, ) @@ -24,25 +23,35 @@ DEFAULT_TARGETS = [ repo=Repository(owner="apache", name="airflow", ref="main"), check_options=CheckOptions(select="ALL"), ), + Project( + repo=Repository(owner="apache", name="superset", ref="master"), + check_options=CheckOptions(select="ALL"), + ), Project(repo=Repository(owner="aws", name="aws-sam-cli", ref="develop")), + Project(repo=Repository(owner="binary-husky", name="gpt_academic", ref="master")), Project(repo=Repository(owner="bloomberg", name="pytest-memray", ref="main")), Project( repo=Repository(owner="bokeh", name="bokeh", ref="branch-3.3"), check_options=CheckOptions(select="ALL"), ), - Project( - repo=Repository(owner="demisto", name="content", ref="master"), - format_options=FormatOptions( - # Syntax errors in this file - exclude="Packs/ThreatQ/Integrations/ThreatQ/ThreatQ.py" - ), - ), + # Disabled due to use of explicit `select` with `E999`, which is no longer + # supported in `--preview`. + # See: https://github.com/astral-sh/ruff/pull/12129 + # Project( + # repo=Repository(owner="demisto", name="content", ref="master"), + # format_options=FormatOptions( + # # Syntax errors in this file + # exclude="Packs/ThreatQ/Integrations/ThreatQ/ThreatQ.py" + # ), + # ), Project(repo=Repository(owner="docker", name="docker-py", ref="main")), + Project(repo=Repository(owner="facebookresearch", name="chameleon", ref="main")), Project(repo=Repository(owner="freedomofpress", name="securedrop", ref="develop")), Project(repo=Repository(owner="fronzbot", name="blinkpy", ref="dev")), Project(repo=Repository(owner="ibis-project", name="ibis", ref="main")), Project(repo=Repository(owner="ing-bank", name="probatus", ref="main")), Project(repo=Repository(owner="jrnl-org", name="jrnl", ref="develop")), + Project(repo=Repository(owner="langchain-ai", name="langchain", ref="master")), Project(repo=Repository(owner="latchbio", name="latch", ref="main")), Project(repo=Repository(owner="lnbits", name="lnbits", ref="main")), Project(repo=Repository(owner="milvus-io", name="pymilvus", ref="master")), @@ -66,6 +75,7 @@ DEFAULT_TARGETS = [ check_options=CheckOptions(select="E,F,FA,I,PYI,RUF,UP,W"), ), Project(repo=Repository(owner="python-poetry", name="poetry", ref="master")), + Project(repo=Repository(owner="qdrant", name="qdrant-client", ref="master")), Project(repo=Repository(owner="reflex-dev", name="reflex", ref="main")), Project(repo=Repository(owner="rotki", name="rotki", ref="develop")), Project(repo=Repository(owner="scikit-build", name="scikit-build", ref="main")), @@ -109,10 +119,10 @@ DEFAULT_TARGETS = [ check_options=CheckOptions(select=JUPYTER_NOTEBOOK_SELECT), config_overrides={ "include": ["*.ipynb"], - # TODO(charlie): Re-enable after fixing typo. + # TODO(dhruvmanila): Re-enable after fixing the notebook. "exclude": [ - "examples/dalle/Image_generations_edits_and_variations_with_DALL-E.ipynb", - "examples/How_to_handle_rate_limits.ipynb", + "examples/gpt_actions_library/.gpt_action_getting_started.ipynb", + "examples/gpt_actions_library/gpt_action_bigquery.ipynb", ], }, ),