Skip to content
This repository was archived by the owner on Oct 11, 2022. It is now read-only.

Conversation

@mxstbr
Copy link
Contributor

@mxstbr mxstbr commented Oct 3, 2018

Status

  • WIP
  • Ready for review
  • Needs testing

  • Runs unit tests before building and starting the API and web app in CI—they aren't needed for the unit tests, and it means if there's an issue with the unit tests we'll know about it more quickly
  • Builds the desktop apps after running e2e tests in CI—the desktop app builds are usually fine, whereas e2e tests fail more often so we want to more quickly know if it happened

mxstbr added 2 commits October 3, 2018 14:00
This should give us quicker feedback if the unit tests have troubles
@brianlovin brianlovin merged commit 2dcca3e into alpha Oct 3, 2018
@brianlovin brianlovin deleted the faster-unit-test-fails branch October 3, 2018 22:41
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants