Skip to content

Conversation

@kesselb
Copy link
Collaborator

@kesselb kesselb commented Jun 10, 2022

tests/drone-run-php-tests.sh is there to reduce CI load a bit. The idea to skip for example the php tests when only vue/js is touched. Screenshot below is from a dependency update. As we don't have a check for 3rdparty the php tests are skipped and CI is happy. But we didn't run the unit tests for the dependency update 🙈

image

Example to test the condition:

[[ $(git diff --name-only 51c61d3194a306c68daab34537742f476408cfe3...aee4d39aa974431bbe473829fc44e7084b43e809 | grep -c "3rdparty") -gt 0 ]] && echo "3rdparty is modified";

Signed-off-by: Daniel Kesselberg <mail@danielkesselberg.de>
@kesselb kesselb force-pushed the debt/noid/check-for-3rdparty-updates branch from 16212b7 to 17a12bf Compare June 10, 2022 12:20
@kesselb kesselb added 3. to review Waiting for reviews technical debt labels Jun 10, 2022
@kesselb kesselb added this to the Nextcloud 25 milestone Jun 10, 2022
@kesselb
Copy link
Collaborator Author

kesselb commented Jun 10, 2022

/backport to stable24

@kesselb
Copy link
Collaborator Author

kesselb commented Jun 10, 2022

/backport to stable23

@kesselb
Copy link
Collaborator Author

kesselb commented Jun 10, 2022

/backport to stable22

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants