Skip to content

[MAINTENANCE] clean up unused code#11717

Open
joshua-stauffer wants to merge 8 commits into
developfrom
m/clean_up_data_context_defaults
Open

[MAINTENANCE] clean up unused code#11717
joshua-stauffer wants to merge 8 commits into
developfrom
m/clean_up_data_context_defaults

Conversation

@joshua-stauffer
Copy link
Copy Markdown
Member

  • Description of PR changes above includes a link to an existing GitHub issue
  • PR title is prefixed with one of: [BUGFIX], [FEATURE], [DOCS], [MAINTENANCE], [CONTRIB], [MINORBUMP]
  • Code is linted - run invoke lint (uses ruff format + ruff check)
  • Appropriate tests and docs have been updated

For more information about contributing, visit our community resources.

After you submit your PR, keep the page open and monitor the statuses of the various checks made by our continuous integration process at the bottom of the page. Please fix any issues that come up and reach out on Slack if you need help. Thanks for contributing!

@netlify
Copy link
Copy Markdown

netlify Bot commented Mar 12, 2026

Deploy Preview for niobium-lead-7998 canceled.

Name Link
🔨 Latest commit 334b09b
🔍 Latest deploy log https://app.netlify.com/projects/niobium-lead-7998/deploys/69c1ad5d9f38830009a86cc5

@codecov
Copy link
Copy Markdown

codecov Bot commented Mar 12, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.73%. Comparing base (f0b29f1) to head (334b09b).
⚠️ Report is 77 commits behind head on develop.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop   #11717      +/-   ##
===========================================
- Coverage    84.59%   83.73%   -0.86%     
===========================================
  Files          473      473              
  Lines        39260    39214      -46     
===========================================
- Hits         33211    32836     -375     
- Misses        6049     6378     +329     
Flag Coverage Δ
3.10 73.52% <ø> (+0.01%) ⬆️
3.10 athena ?
3.10 aws_deps ?
3.10 big ?
3.10 clickhouse ?
3.10 filesystem ?
3.10 mysql ?
3.10 openpyxl or pyarrow or project or sqlite or aws_creds ?
3.10 postgresql ?
3.10 spark ?
3.10 spark_connect ?
3.10 sql_server ?
3.10 trino ?
3.11 73.55% <ø> (-0.01%) ⬇️
3.11 athena ?
3.11 aws_deps ?
3.11 big ?
3.11 clickhouse ?
3.11 filesystem ?
3.11 mysql ?
3.11 openpyxl or pyarrow or project or sqlite or aws_creds ?
3.11 postgresql ?
3.11 spark_connect ?
3.11 sql_server ?
3.11 trino ?
3.12 73.55% <ø> (-0.01%) ⬇️
3.12 athena ?
3.12 aws_deps ?
3.12 big ?
3.12 filesystem ?
3.12 mysql ?
3.12 openpyxl or pyarrow or project or sqlite or aws_creds ?
3.12 postgresql ?
3.12 spark ?
3.12 spark_connect ?
3.12 sql_server ?
3.12 trino ?
3.13 73.57% <ø> (+0.01%) ⬆️
3.13 athena 41.93% <ø> (-0.03%) ⬇️
3.13 aws_deps 45.18% <ø> (-0.03%) ⬇️
3.13 big 55.24% <ø> (-0.01%) ⬇️
3.13 bigquery 51.23% <ø> (-0.02%) ⬇️
3.13 clickhouse 41.94% <ø> (-0.03%) ⬇️
3.13 databricks 53.03% <ø> (-0.02%) ⬇️
3.13 filesystem 64.33% <ø> (+<0.01%) ⬆️
3.13 gx-redshift 51.38% <ø> (-0.02%) ⬇️
3.13 mysql 51.78% <ø> (-0.02%) ⬇️
3.13 openpyxl or pyarrow or project or sqlite or aws_creds 59.94% <ø> (-0.01%) ⬇️
3.13 postgresql 55.18% <ø> (-0.01%) ⬇️
3.13 snowflake 53.87% <ø> (-0.02%) ⬇️
3.13 spark 55.88% <ø> (-0.02%) ⬇️
3.13 spark_connect 46.84% <ø> (-0.02%) ⬇️
3.13 sql_server 53.21% <ø> (-0.02%) ⬇️
3.13 trino 48.72% <ø> (-0.02%) ⬇️
cloud ?
docs-basic ?
docs-creds-needed ?
docs-spark ?

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.

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Copy Markdown
Contributor

@NathanFarmer NathanFarmer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@github-actions
Copy link
Copy Markdown
Contributor

Is this PR still relevant? If so, what is blocking it? Is there anything you can do to help move it forward?

This issue has been automatically marked as stale because it has not had recent activity.

It will be closed if no further activity occurs. Thank you for your contributions 🙇

@github-actions github-actions Bot added stale Stale issues and PRs and removed stale Stale issues and PRs labels Apr 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants