Powershell: Tests testing that all packages use the latest version should not be part of CI build

Created on 11 Oct 2018  路  7Comments  路  Source: PowerShell/PowerShell

PR #7661 added package.tests.ps1, to check that all referenced packages are of the latest version. This causes a lot of unnecessary build failures in forks and PRs because it takes time for the bot to kick in and the update to be approved and merged and then one needs to upstream or re trigger the build and hope that no other 3rd party has published a new update in the meantime...
Those tests should be moved to a different build definition that is run e.g. only on a daily basis but definitely not part of CI or a PR. cc @TravisEz13

Resolution-Fixed WG-Quality-Test
Source
picture bergmeister

All 7 comments

Agreed. We have added a bot to do this work now. The tests can be removed or disabled.

TravisEz13 picture TravisEz13 on 11 Oct 2018
👍1

@iSazonov I do not see this issue as closed until the tests are not part of the CI/PR builds any more

bergmeister picture bergmeister on 11 Oct 2018

@bergmeister Sorry.

iSazonov picture iSazonov on 11 Oct 2018
👍1

I'll submit PR to remove these tests

SteveL-MSFT picture SteveL-MSFT on 11 Oct 2018
👍1

Fixed

TravisEz13 picture TravisEz13 on 11 Oct 2018

Thanks :-)

bergmeister picture bergmeister on 12 Oct 2018

No problem, we were just try to find the right solution. Sometimes it takes trial and error.

TravisEz13 picture TravisEz13 on 12 Oct 2018
👍1
Was this page helpful?
0 / 5 - 0 ratings

Related issues

HumanEquivalentUnit picture HumanEquivalentUnit  路  3Comments
alx9r picture alx9r  路  3Comments
JohnLBevan picture JohnLBevan  路  3Comments
ajensenwaud picture ajensenwaud  路  3Comments
concentrateddon picture concentrateddon  路  3Comments