Skip to content

chore(deps): require manual deck.gl upgrades - #42653

Open
sadpandajoe wants to merge 1 commit into
masterfrom
codex/fix-dependabot-overrides-7d2b184
Open

chore(deps): require manual deck.gl upgrades#42653
sadpandajoe wants to merge 1 commit into
masterfrom
codex/fix-dependabot-overrides-7d2b184

Conversation

@sadpandajoe

Copy link
Copy Markdown
Member

SUMMARY

Exclude the coupled @deck.gl/* and @luma.gl/* package families from automated
npm updates. These dependencies span the frontend root and plugin workspaces and
are also pinned by root overrides, so they require a single manually validated
upgrade.

The master Dependabot run produced 13 npm EOVERRIDE errors while trying to
update direct dependencies whose versions are duplicated by literal overrides.
Using self-referencing overrides removes that immediate error but exposes
incompatible peer ranges across the workspaces, so it is not a safe automated
upgrade path. The ignore rules prevent these two failure families without
suppressing unrelated npm updates.

Observed failure: https://github.com/apache/superset/actions/runs/30611456387

BEFORE/AFTER SCREENSHOTS OR ANIMATED GIF

Not applicable; Dependabot configuration only.

TESTING INSTRUCTIONS

  • pre-commit run --files .github/dependabot.yml
  • pre-commit run --all-files was run. YAML and zizmor passed; unrelated master MyPy failures plus local frontend/docs dependency gaps prevented the repository-wide command from completing cleanly.
  • Reproduced the original conflict with npm install @deck.gl/core@9.3.7 --package-lock-only --dry-run=true --ignore-scripts from superset-frontend.
  • Confirm the next scheduled Dependabot run completes without deck.gl/luma.gl updater errors and still proposes unrelated dependency updates.

ADDITIONAL INFORMATION

  • Has associated issue:
  • Required feature flags:
  • Changes UI
  • Includes DB Migration (follow approval process in SIP-59)
    • Migration is atomic, supports rollback & is backwards-compatible
    • Confirm DB migration upgrade and downgrade tested
    • Runtime estimates and downtime expectations provided
  • Introduces new feature or API
  • Removes existing feature or API

@netlify

netlify Bot commented Jul 31, 2026

Copy link
Copy Markdown

Deploy Preview for superset-docs-preview ready!

Name Link
🔨 Latest commit d371cf5
🔍 Latest deploy log https://app.netlify.com/projects/superset-docs-preview/deploys/6a6ce16661223000087598f8
😎 Deploy Preview https://deploy-preview-42653--superset-docs-preview.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.
🤖 Make changes Run an agent on this branch

To edit notification comments on pull requests, go to your Netlify project configuration.

@sadpandajoe

Copy link
Copy Markdown
Member Author

@rusackas hopefully this helps, but feel free to close if we think it won't be useful and only cause more issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant