dev: enable ruff rule

[anshbansal]

Checklist

• The PR conforms to DataHub's Contributing Guideline (particularly Commit Message Format)
• Links to related issues (if applicable)
• Tests for the changes have been added/updated (if applicable)
• Docs related to the changes have been added/updated (if applicable). If a new feature has been added a Usage Guide has been added for the same.
• For any breaking change/potential downtime/deprecation/big changes an entry has been made in Updating DataHub

[codecov]

Codecov Report

Attention: Patch coverage is 78.00000% with 11 lines in your changes missing coverage. Please review.

✅ All tests successful. No failed tests found.

Files with missing lines	Patch %	Lines
...gestion/src/datahub/ingestion/source/abs/config.py	0.00%	2 Missing ⚠️
...stion/src/datahub/ingestion/source/sql/teradata.py	0.00%	2 Missing ⚠️
...ta-ingestion/src/datahub/ingestion/graph/client.py	0.00%	1 Missing ⚠️
...hub/ingestion/source/bigquery_v2/bigquery_audit.py	0.00%	1 Missing ⚠️
...ub/ingestion/source/bigquery_v2/bigquery_schema.py	0.00%	1 Missing ⚠️
...datahub/ingestion/source/gc/dataprocess_cleanup.py	0.00%	1 Missing ⚠️
...tion/src/datahub/ingestion/source/sql/sql_types.py	50.00%	1 Missing ⚠️
metadata-ingestion/src/datahub/lite/duckdb_lite.py	0.00%	1 Missing ⚠️
metadata-ingestion/src/datahub/sdk/dataset.py	0.00%	1 Missing ⚠️

📢 Thoughts on this report? Let us know!

[sgomezvillamor]

Overall looks good

Just concern on:

• why some fixes have been applied for rules that we are ignoring, as per my understanding
• default args becoming Optional

[sgomezvillamor]

While in practice this is not a breaking change, I prefer to keep the default value in the signature than inside the implementation. Because knowing the default value is relevant for the caller.

[anshbansal]

mutable args can lead to bugs https://docs.python-guide.org/writing/gotchas/ This is a well documented problem.