With my python linter, I detected some issues with the codebase.
I will list them here:
trigger_airbyte_sync not defined here:
|
return trigger_airbyte_sync |
database_info can send an error or an empty table, we should add a quick check after any MTB request (debugging will be easier later) :
|
database_info = get_database_info(database_id) |
- same thing here:
|
new_card_id = copy_card_to_collection(ref_card["id"], target_collection_id, new_name=card_name) |
decidim_obj cannot be none in this function:
|
_decidim_obj(ns="b", name="z", host="z.example", ready_status="True", message="ok", ltt="t3", status_version=None, image="app"), |
. If it can be, the function should say it
parse does not exist in urllib:
|
values["url_param_date"] = urllib.parse.quote(values["schedule_date"]) |
With my python linter, I detected some issues with the codebase.
I will list them here:
trigger_airbyte_syncnot defined here:airflow-orchestration/dags/main_orchestration.py
Line 61 in ca465de
database_infocan send an error or an empty table, we should add a quick check after any MTB request (debugging will be easier later) :airflow-orchestration/dags/data_utils/metabase_automation/metabase_automation.py
Line 59 in ca465de
airflow-orchestration/dags/data_utils/metabase_automation/metabase_airflow_task.py
Line 110 in ca465de
decidim_objcannot be none in this function:airflow-orchestration/dags/data_utils/grist/test_decidim_info.py
Line 101 in ca465de
parsedoes not exist inurllib:airflow-orchestration/dags/data_utils/alerting/alerting.py
Line 18 in ca465de