forked from DonJayamanne/pythonVSCode
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
Closed
Labels
debtCovers everything internal: CI, testing, refactoring of the codebase, etc.Covers everything internal: CI, testing, refactoring of the codebase, etc.
Description
The core contributors of the extension have to wait for the tests to pass before we can merge the tests (sometimes the tests take forever, sometimes they need to be re-run - due to flaky tests).
All of this slows down the time taken to merge a PR.
Solution:
- Run tests on VSTS (all platforms)
- Once tests pass, merge the PR
- If tests fail on Master (AppVeyor or Travis), then
- Restart tests
- Worst case, identify the problem (identify why tests pass on VSTS and not on AppVeyor/Travis)
- No changes to external contributors
@brettcannon @MikhailArkhipov
Thoughts
Metadata
Metadata
Assignees
Labels
debtCovers everything internal: CI, testing, refactoring of the codebase, etc.Covers everything internal: CI, testing, refactoring of the codebase, etc.