Make Resin-SDK builds gentler on the API about balena-sdk HOT 6 CLOSED

@emirotin @jviotti @Page- I think we should:

• Stop building on pushes and instead just build on PRs

• Find a way to change the NPM config + build config so we only run the test suite once per job. I've got a plan for a reasonably nice solution to this that doesn't cause problems for pre-NPM4 users - I'll push a strawman in a minute.

Thoughts?

from balena-sdk.

from balena-sdk.

#242 is a run at a fix for the NPM issue here, and I've now disabled building on pushes for Travis. I can't see how to get at the equivalent settings for AppVeyor though, @jviotti I think you've got access - if you could change that to only build on PRs too that'd be great, thanks.

from balena-sdk.

Hi @pimterry ,

I made sure you have access to Appveyor by including it into the CLI/SDK GH team (which is explicitly added to Appveyor). After looking a bit around, I think the only way to accomplish what you're trying to do is by adding the following to appveyor.yml:

branches:
  only:
    - master

This will prevent builds for branches, but it looks that it will still build PRs (according to the docs). Give that a go and let me know if it still doesn't work, and we can get in touch with Appveyor's priority support.

from balena-sdk.

@jviotti Thanks! That pointer was enough to get me to the config option I was looking for: appveyor/ci#882. #244 turns this on.

Sorry for the PR flood! With that one though we should hopefully have most of these build issues in the SDK solved, for now 😄

from balena-sdk.

All done - we now run 25% of the tests we were running before, so I'm going to close this. Should drastically improve things. If we need to go further in future, we'll need to substantially refactor the actual flow of the tests, but I suggest we leave that unless we see more problems that'd make it necessary.

from balena-sdk.