svrooij commented 3 years ago

Sonos notification queue

Description

This will 'queue' notifications when started before the other one is finished. Which allows you to play multiple notifications by just calling play notification multiple times.

Your checklist for this pull request

🚨 Please review the guidelines for contributing to this repository.

[x] Make sure you are requesting to pull a topic/feature/bugfix branch (right side). Don't request your master!
[x] Make sure you are making a pull request against the beta branch (left side). Also you should start your branch off svrooij/node-sonos-ts/beta.
[x] Check the commit's or even all commits' message styles matches our requested structure.
[x] Check your code additions will fail neither code linting checks nor unit test.
[x] Check you add tests for added code.

💔 Thank you!

coveralls commented 3 years ago

Pull Request Test Coverage Report for Build 493882988

65 of 67 (97.01%) changed or added relevant lines in 1 file are covered.
No unchanged relevant lines lost coverage.
Overall coverage increased (+0.4%) to 78.351%

Changes Missing Coverage	Covered Lines	Changed/Added Lines	%
src/sonos-device.ts	65	67	97.01%
<!--	Total:	65	67	97.01%	-->

Totals
Change from base Build 493691840:	0.4%
Covered Lines:	1274
Relevant Lines:	1643

💛 - Coveralls

github-actions[bot] commented 3 years ago

:tada: This PR is included in version 2.4.0-beta.1 :tada:

The release is available on:

Your semantic-release bot :package::rocket: