Skip to content

Conversation

@miloszwielgus
Copy link
Collaborator

@miloszwielgus miloszwielgus commented Oct 1, 2025

Closes RNAA-264

⚠️ Breaking changes ⚠️

Introduced changes

Implemented ConstantSourceNode

Checklist

  • Linked relevant issue
  • Updated relevant documentation
  • Added/Conducted relevant tests
  • Performed self-review of the code
  • Updated Web Audio API coverage
  • Added support for web

@miloszwielgus miloszwielgus marked this pull request as ready for review October 1, 2025 13:22
Copy link
Contributor

@poneciak57 poneciak57 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@vltkv vltkv left a comment

Choose a reason for hiding this comment

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

Great work!

Copy link
Member

@michalsek michalsek left a comment

Choose a reason for hiding this comment

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

Besides existing comments LGTM, good job! convolver next :)

Copy link
Collaborator

@maciejmakowski2003 maciejmakowski2003 left a comment

Choose a reason for hiding this comment

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

great work. congrats for first core contribution!

@maciejmakowski2003 maciejmakowski2003 merged commit 96fd676 into software-mansion:main Oct 3, 2025
3 checks passed
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.

5 participants