Skip to content

fix: use @chainable per-overload#8504

Merged
davepagurek merged 2 commits intoprocessing:dev-2.0from
LuLaValva:8356-chainable-ts
Feb 8, 2026
Merged

fix: use @chainable per-overload#8504
davepagurek merged 2 commits intoprocessing:dev-2.0from
LuLaValva:8356-chainable-ts

Conversation

@LuLaValva
Copy link
Copy Markdown

Changes:

  • @chainable in JSDoc applies to one overload and not the whole function
  • To test, view updates to p5.d.ts after npm run generate-types
code screenshot showing that the type signature for a size method changed from returning Element for both overloads to object and element in the correct places

Updated methods: volume, speed, position, size, style, attribute, value

PR Checklist

I don't think any of these apply 😅... lint has a bunch of warnings but none related to my code, and I don't see unit/snapshot tests for the p5.d.ts file.

Copy link
Copy Markdown
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.

Awesome, thanks for the quick fix!

@davepagurek
Copy link
Copy Markdown
Contributor

@all-contributors please add @LuLaValva for bug, code

@davepagurek davepagurek merged commit 5dbbab9 into processing:dev-2.0 Feb 8, 2026
2 checks passed
@ksen0 ksen0 added this to the 2.3 milestone Feb 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Completed

Development

Successfully merging this pull request may close these issues.

3 participants