[MAINTENANCE] Ban future use of convert_to_json_serializable

[Kilo59]

Use of convert_to_json_serializable is a sign that we aren't properly using pydantic models for objects that require json serialization.

Ban the import of the function and add # noqa: TID25 for existing violations.

Also ban ensure_json_serializable (maybe we don't care as much about this one?)

[netlify]

✅ Deploy Preview for niobium-lead-7998 canceled.

Name	Link
🔨 Latest commit	fec2dec
🔍 Latest deploy log	https://app.netlify.com/sites/niobium-lead-7998/deploys/664544ffed753300082dc41b

[codecov]

Codecov Report

Attention: Patch coverage is 96.42857% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 77.85%. Comparing base (499430b) to head (fec2dec).

Files	Patch %	Lines
...rule_based_profiler/helpers/runtime_environment.py	0.00%	1 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff            @@
##           develop    #9935   +/-   ##
========================================
  Coverage    77.85%   77.85%           
========================================
  Files          454      454           
  Lines        39517    39517           
========================================
  Hits         30766    30766           
  Misses        8751     8751           

Flag	Coverage Δ
3.10	64.39% <96.42%> (+0.01%)	⬆️
3.11	64.38% <96.42%> (ø)
3.11 athena or clickhouse or openpyxl or pyarrow or project or sqlite or aws_creds	53.62% <96.42%> (ø)
3.11 aws_deps	44.79% <96.42%> (ø)
3.11 big	53.84% <96.42%> (ø)
3.11 databricks	46.07% <96.42%> (ø)
3.11 filesystem	59.33% <96.42%> (ø)
3.11 mssql	49.06% <96.42%> (ø)
3.11 mysql	49.12% <96.42%> (ø)
3.11 postgresql	53.25% <96.42%> (ø)
3.11 snowflake	46.74% <96.42%> (+<0.01%)	⬆️
3.11 spark	56.52% <96.42%> (ø)
3.11 trino	51.16% <96.42%> (ø)
3.8	64.42% <96.42%> (ø)
3.8 athena or clickhouse or openpyxl or pyarrow or project or sqlite or aws_creds	53.62% <96.42%> (ø)
3.8 aws_deps	44.81% <96.42%> (ø)
3.8 big	53.85% <96.42%> (ø)
3.8 databricks	46.09% <96.42%> (ø)
3.8 filesystem	59.34% <96.42%> (ø)
3.8 mssql	49.04% <96.42%> (ø)
3.8 mysql	49.10% <96.42%> (ø)
3.8 postgresql	53.24% <96.42%> (ø)
3.8 snowflake	46.76% <96.42%> (ø)
3.8 spark	56.48% <96.42%> (ø)
3.8 trino	51.15% <96.42%> (ø)
3.9	64.41% <96.42%> (ø)
cloud	0.00% <0.00%> (ø)
docs-basic	47.62% <57.14%> (ø)
docs-creds-needed	48.74% <57.14%> (ø)
docs-spark	47.47% <57.14%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[Kilo59]

Not sure if we care as much about ensure_json_serializable 🤷

[jcampbell]

Yes, this is what I had in mind!