Skip to content

docs(concat): clarify array inputs#7605

Open
beanscg wants to merge 1 commit into
ReactiveX:masterfrom
beanscg:bean-labs-concat-array-doc-example
Open

docs(concat): clarify array inputs#7605
beanscg wants to merge 1 commit into
ReactiveX:masterfrom
beanscg:bean-labs-concat-array-doc-example

Conversation

@beanscg

@beanscg beanscg commented Jun 8, 2026

Copy link
Copy Markdown

Fixes #4697.

This updates the concat documentation to remove the misleading statement that an array of Observables can be passed directly. Since concat accepts rest arguments, passing an array emits the Observable objects in that array; callers that already have an array should spread it with concat(...timers).

Changes:

  • Clarify that each Observable should be passed as its own argument.
  • Explain the array-spread behavior and the failure mode of concat(timers).
  • Add an array-of-Observables example using concat(...timers).

Validation:

  • git diff --check
  • Node content check for the updated warning and concat(...timers) example

Full docs/test suite was not run locally because this environment does not have this repository's dependencies installed.

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.

concat example does not work with an array of Observables

1 participant