Skip to content

[HOLD] feat(components/packages)!: remove @skyux/packages/polyfills#4235

Draft
Blackbaud-SteveBrush wants to merge 5 commits intomainfrom
remove-polyfills
Draft

[HOLD] feat(components/packages)!: remove @skyux/packages/polyfills#4235
Blackbaud-SteveBrush wants to merge 5 commits intomainfrom
remove-polyfills

Conversation

@Blackbaud-SteveBrush
Copy link
Copy Markdown
Member

@Blackbaud-SteveBrush Blackbaud-SteveBrush commented Feb 24, 2026

Hold reason: We have a handful of consumers that still use dragula directly; they'll likely need this polyfill for their tests. Investigating options...

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai Bot commented Feb 24, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch remove-polyfills

Tip

Try Coding Plans. Let us write the prompt for your AI agent so you can ship faster (with fewer bugs).
Share your feedback on Discord.


Comment @coderabbitai help to get the list of available commands and usage tips.

@nx-cloud
Copy link
Copy Markdown

nx-cloud Bot commented Feb 24, 2026

View your CI Pipeline Execution ↗ for commit 4bd590e

Command Status Duration Result
nx build code-examples-playground --baseHref=ht... ✅ Succeeded 13s View ↗
nx build integration --baseHref=https://blackba... ✅ Succeeded 10s View ↗
nx build playground --baseHref=https://blackbau... ✅ Succeeded 8s View ↗

☁️ Nx Cloud last updated this comment at 2026-02-25 13:59:46 UTC

@Blackbaud-SteveBrush Blackbaud-SteveBrush added risk level (author): 2 This change has a slight chance of introducing a bug labels Feb 24, 2026
@Blackbaud-SteveBrush Blackbaud-SteveBrush changed the title feat(components/packages)!: remove @skyux/packages/polyfills [HOLD] feat(components/packages)!: remove @skyux/packages/polyfills Feb 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk level (author): 2 This change has a slight chance of introducing a bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants