Skip to content

Conversation

@driazati
Copy link
Member

@driazati driazati commented May 25, 2022

This improves the parsing to avoid issues like in #11454

cc @Mousius @areusch

@Lunderberg
Copy link
Contributor

Ooh, nice. So that was triggered by the empty square brackets in the PR title?

@driazati
Copy link
Member Author

Ooh, nice. So that was triggered by the empty square brackets in the PR title?

right, that plus some bad parsing logic of the source teams issue (more details in the logs), I'd like to clean it up a bit but I don't have permissions to edit other people's comments out

This improves the parsing to avoid issues like in #11454

commit-id:53a06ab3
@driazati driazati changed the title [skip ci][ci] Add filter to teams [ci] Add filter to teams May 27, 2022
@driazati driazati marked this pull request as ready for review May 27, 2022 23:21
@driazati driazati marked this pull request as draft June 1, 2022 15:31
@driazati driazati marked this pull request as ready for review June 1, 2022 15:32
@github-actions github-actions bot requested review from Mousius and areusch June 1, 2022 15:34
@driazati
Copy link
Member Author

driazati commented Jun 1, 2022

@tvm-bot merge

@github-actions
Copy link
Contributor

github-actions bot commented Jun 1, 2022

Cannot merge, these CI jobs are not successful on 8b000d0:

@areusch
Copy link
Contributor

areusch commented Jun 1, 2022

@tvm-bot merge

@github-actions
Copy link
Contributor

github-actions bot commented Jun 1, 2022

Cannot merge, these CI jobs are not successful on 8b000d0:

@areusch areusch merged commit ce60bfa into apache:main Jun 1, 2022
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.

3 participants