Skip to content

Remove merge queue trigger and update dependencies#563

Open
jamesrweb wants to merge 3 commits intomainfrom
chore/remove-merge-queue-and-update-deps
Open

Remove merge queue trigger and update dependencies#563
jamesrweb wants to merge 3 commits intomainfrom
chore/remove-merge-queue-and-update-deps

Conversation

@jamesrweb
Copy link
Copy Markdown
Member

Related Issue

Fixes # N/A — maintenance task with no corresponding issue.

PR Type

  • 🐛 Bug Fix
  • ✨ New Feature
  • 🔨 Code Refactor
  • 📝 Documentation Update
  • 🧪 Test Update
  • 🔧 Build/CI Update
  • 🧹 Chore
  • ⏪ Revert

Description

The merge queue has been removed from the main branch protection rules. This PR removes the now-unnecessary merge_group event trigger from the CI workflow and updates all project dependencies to their latest compatible versions. This also closes and supersedes the open dependabot PRs (#559, #561, #562) which were failing due to the merge queue blocking auto-merge.

Proposed Changes

  • Removed merge_group trigger from .github/workflows/continuous-integration.yml
  • Updated all project dependencies to latest compatible versions via npm-check-updates
  • TypeScript 6.x excluded as typescript-eslint does not yet support it

How Has This Been Tested?

  • Unit Tests
  • Integration Tests
  • Manual Testing (please describe)

CI will validate formatting, linting, tests, and build on this PR. The CI workflow change is a trigger removal only, with no impact on job logic.

Screenshots/Recordings

N/A

Breaking Changes

  • Yes (please describe)
  • No

Checklist

  • My code follows the code style of this project
  • I have updated the documentation accordingly
  • I have added tests to cover my changes
  • All new and existing tests passed
  • My changes generate no new warnings

Additional Notes

This PR supersedes and replaces the following dependabot PRs which were closed:

The dependabot workflow's gh pr merge --auto --merge step was failing with Pull request User is not authorized for this protected branch (enablePullRequestAutoMerge) because the merge queue restricted which tokens could call that GraphQL mutation. With the merge queue removed, future dependabot PRs should auto-merge successfully.

🤖 Generated with Claude Code

jamesrweb and others added 2 commits March 30, 2026 01:48
The merge queue has been removed from the main branch protection rules,
so the merge_group event trigger is no longer needed.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Updated the following dependencies:
- @typescript-eslint/eslint-plugin: ^8.57.1 → ^8.57.2
- @vitest/coverage-v8: ^4.1.0 → ^4.1.2
- happy-dom: ^20.8.4 → ^20.8.9
- pnpm: 10.32.1 → 10.33.0
- typescript-eslint: ^8.57.1 → ^8.57.2
- vite: ^8.0.1 → ^8.0.3
- vitest: ^4.1.0 → ^4.1.2
- vitest-canvas-mock: ^1.1.3 → ^1.1.4

TypeScript 6.x was excluded as typescript-eslint does not yet support it.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@jamesrweb jamesrweb requested a review from yevdyko as a code owner March 29, 2026 23:51
@jamesrweb jamesrweb self-assigned this Mar 29, 2026
@jamesrweb jamesrweb added enhancement dependencies Pull requests that update a dependency file github-actions labels Mar 29, 2026
@github-actions
Copy link
Copy Markdown

github-actions bot commented Mar 29, 2026

Coverage report for commit: 060c637
File: ./coverage/clover.xml

Cover ┌─────────────────────────┐ Freq.
   0% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  10% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  20% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  30% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  40% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  50% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  60% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  70% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  80% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
  90% │ ░░░░░░░░░░░░░░░░░░░░░░░ │  0.0%
 100% │ ███████████████████████ │ 100.0%
      └─────────────────────────┘
 *Legend:* █ = Current Distribution 
Summary - Lines: 100.00% | Methods: 100.00% | Branches: 97.22%
FilesLinesMethodsBranches
src/components
   P5Canvas.tsx100.00%100.00%100.00%
   P5CanvasGuard.tsx100.00%100.00%87.50%
   P5CanvasWithSketch.tsx100.00%100.00%100.00%
src/constants
   CanvasContainerClassName.ts100.00%100.00%100.00%
src/contracts
   CanvasContainer.ts100.00%100.00%100.00%
   CanvasContainerRef.ts100.00%100.00%100.00%
   InputProps.ts100.00%100.00%100.00%
   P5CanvasInstance.ts100.00%100.00%100.00%
   P5CanvasInstanceRef.ts100.00%100.00%100.00%
   P5CanvasProps.ts100.00%100.00%100.00%
   P5CanvasPropsWithSketch.ts100.00%100.00%100.00%
   Sketch.ts100.00%100.00%100.00%
   SketchProps.ts100.00%100.00%100.00%
   WithChildren.ts100.00%100.00%100.00%
   p5.ts100.00%100.00%100.00%
src
   main.tsx100.00%100.00%100.00%
src/utils
   createP5CanvasInstance.ts100.00%100.00%100.00%
   logErrorBoundaryError.ts100.00%100.00%100.00%
   propsAreEqual.ts100.00%100.00%100.00%
   removeP5CanvasInstance.ts100.00%100.00%100.00%
   updateP5CanvasInstance.ts100.00%100.00%100.00%
   withoutKeys.ts100.00%100.00%100.00%

🤖 comment via lucassabreu/comment-coverage-clover

@jamesrweb jamesrweb changed the title chore: remove merge queue trigger and update dependencies Remove merge queue trigger and update dependencies Mar 29, 2026
* chore: migrate tsconfig.json for TypeScript 6 compatibility

- Remove deprecated `baseUrl` and prefix all `paths` entries with `./`
- Remove `allowSyntheticDefaultImports` (always enabled in TS6)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

* chore(deps): upgrade to TypeScript 6 and typescript-eslint 8.57.3-alpha.3

- TypeScript: ^5.9.3 → ^6.0.2
- typescript-eslint: ^8.57.2 → 8.57.3-alpha.3
- @typescript-eslint/eslint-plugin: ^8.57.2 → 8.57.3-alpha.3

The alpha release of typescript-eslint is required as the stable 8.57.2
does not yet include TypeScript 6 peer dependency support (PRs merged,
pending release).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

---------

Co-authored-by: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Copy link
Copy Markdown
Contributor

@yevdyko yevdyko left a comment

Choose a reason for hiding this comment

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

Good stuff! Thanks @jamesrweb 🚀

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

Labels

dependencies Pull requests that update a dependency file enhancement github-actions

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants