bencevans / node-sonos

πŸ”ˆ Sonos Media Player Interface/Client

Home Page:https://www.npmjs.com/package/sonos

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Should we setup 'beta' release from new beta branch?

svrooij opened this issue Β· comments

Should we have a beta branch with a beta release?

A lot of libraries have a special branch setup do to small beta releases that get pulled to master once in a while. Do you think node-sonos should also implement that strategy?

  • By having a beta release we can test PR's more easily, by just releasing something to beta, and have people test it out. Beta suggest that things might break when using the beta release.
  • By having a beta release it should be easier to release big changes. This library doesn't have a high test coverage, so having a beta would allow other contributors to get their changes merged quicker.

Votes

  • πŸ‘ For Yeah, you should!!
  • πŸ˜• For, I don't know, need more info
  • πŸ‘Ž For no, why would you need that

Also comment your thoughts please.

Related to:

I think we could spend some time working on the test coverage. Right now there's like one big file, and it looks like it'd be pretty hard to make changes in there. Could we maybe break the tests up? I'd suggest a convention of tests stored close to the relevant files, i.e. in __test__ folders. Personal choice would be switching to jest?

This poll wasn't about test coverage, but you're right about the test part.

I think we could spend some time working on the test coverage. Right now there's like one big file, and it looks like it'd be pretty hard to make changes in there. Could we maybe break the tests up? I'd suggest a convention of tests stored close to the relevant files, i.e. in __test__ folders. Personal choice would be switching to jest?

  • Folder, no opinion, I'll follow
  • Jest, yes please. In my other sonos library I've also switched to Jest, I really like the build-in code coverage check that you can see if you open Run tests here.
  • Jest also allows easy integration with coveralls to check which parts of the code are checked.
  • Using jest would probably allow us to copy these tests, since they will both test the same code being executed.

This poll wasn't about test coverage

This was meant as a comment to your original point:

This library doesn't have a high test coverage, so having a beta would allow other contributors to get their changes merged quicker.

I'm all for a beta branch. But if contributions are not happening because of lack of test coverage, I'd be willing to put in some work to increase that also. That's all.

I've created a alpha branch but the NPM_TOKEN still has to be setup see #480

Since @bencevans added the NPM secret we can now release much faster.
Choose for auto releases from the alpha branch for now, and then once in a while we push all the changes from alpha to master.

The alpha release might break sometime!!