Skip to content

Conversation

@nbogie
Copy link
Contributor

@nbogie nbogie commented Jan 13, 2026

Resolves #8409

Changes:

  1. Added to patch.mjs to replace type for shuffle(), now:
shuffle<T>(array: T[], bool?: boolean): T[];
  1. Added a test for this type to /test/types/generics.ts

PR Checklist

  • npm run lint passes - NO. Has previous errors.
  • [Unit tests] are included / updated
  • npm run test:types passes - NO. (There are previous errors in test/types/webgl-pixels.ts and test/types/typography.ts)

Copy link
Contributor

@davepagurek davepagurek left a comment

Choose a reason for hiding this comment

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

Thanks for adding the fix!

@davepagurek davepagurek merged commit e5566ec into processing:dev-2.0 Jan 13, 2026
2 checks passed
@nbogie nbogie deleted the fix-types-for-shuffle branch January 13, 2026 20:12
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