Skip to content

Move needs up in reusable-testing.yml#91

Merged
swissspidy merged 1 commit intowp-cli:mainfrom
wojsmol:update/move-needs-up
Nov 10, 2023
Merged

Move needs up in reusable-testing.yml#91
swissspidy merged 1 commit intowp-cli:mainfrom
wojsmol:update/move-needs-up

Conversation

@wojsmol
Copy link
Copy Markdown
Contributor

@wojsmol wojsmol commented Nov 10, 2023

To avoid generatin a job when matrix if empty.

@wojsmol wojsmol requested a review from a team as a code owner November 10, 2023 14:51
@swissspidy
Copy link
Copy Markdown
Member

Still the same?

Screenshot 2023-11-10 at 15 51 46

@wojsmol
Copy link
Copy Markdown
Contributor Author

wojsmol commented Nov 10, 2023

in testing workflow we are using state from main branch so this is expected.

@swissspidy swissspidy changed the title move needs up in reusaqble-testing.yml Move needs up in reusable-testing.yml Nov 10, 2023
Copy link
Copy Markdown
Member

@swissspidy swissspidy left a comment

Choose a reason for hiding this comment

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

Let's give it a try 👍

@swissspidy swissspidy merged commit c28c379 into wp-cli:main Nov 10, 2023
@wojsmol wojsmol deleted the update/move-needs-up branch November 10, 2023 15:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants