Skip to content

[Jenkins] Move each test into its own step#26873

Merged
DeepDiver1975 merged 2 commits into
masterfrom
restructure-jenkinsfile
Jan 1, 2017
Merged

[Jenkins] Move each test into its own step#26873
DeepDiver1975 merged 2 commits into
masterfrom
restructure-jenkinsfile

Conversation

@DeepDiver1975

Copy link
Copy Markdown
Member

Allows us to easier spot the failing test suite

@PVince81 @PhilippSchaffrath

@PVince81 PVince81 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hmm, okay, makes sense 👍

Comment thread Jenkinsfile
export NOCOVERAGE=1
unset USEDOCKER
phpenv local 7.1
make test-php TEST_DATABASE=sqlite

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

some tab vs space madness

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

all spaces now in here .....

@DeepDiver1975 DeepDiver1975 merged commit 8f49009 into master Jan 1, 2017
@DeepDiver1975 DeepDiver1975 deleted the restructure-jenkinsfile branch January 1, 2017 09:57
@PVince81

Copy link
Copy Markdown
Contributor

@DeepDiver1975 can we backport this to stable9.1 and stable9 so we can also backport the other "blow up the test matrix" tests from #26844 ?

@PVince81

Copy link
Copy Markdown
Contributor

s/we/you/ 😉

@PVince81

PVince81 commented Jan 19, 2017

Copy link
Copy Markdown
Contributor

stable9.1: #26976
stable9: #26985

@lock

lock Bot commented Aug 4, 2019

Copy link
Copy Markdown

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@lock lock Bot locked as resolved and limited conversation to collaborators Aug 4, 2019
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.

2 participants