Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CI. Exclude from run build/tests jobs if PR have [WIP]/[Dependent] in the title or has "Draft" status. #3806

Conversation

dvkruchinin
Copy link
Contributor

Motivation and context

These changes allow to exclude the launch unit tests, e2e tests jobs if the PR title starts with [WIP] or [Dependent] or has Draft status.
It should be borne in mind that changing the title of the PR or transferring it to the status "ready for review" does not restart jobs.

How has this been tested?

Checklist

License

  • I submit my code changes under the same MIT License that covers the project.
    Feel free to contact the maintainers if that's a concern.
  • I have updated the license header for each file (see an example below)
# Copyright (C) 2021 Intel Corporation
#
# SPDX-License-Identifier: MIT

@dvkruchinin dvkruchinin requested a review from nmanovic as a code owner October 18, 2021 10:48
@azhavoro
Copy link
Contributor

It should be borne in mind that changing the title of the PR or transferring it to the status "ready for review" does not restart jobs.

@dvkruchinin Hi, is it possible to use action types like edited or ready_for_review to get around this limitation?https://docs.github.com/en/actions/learn-github-actions/events-that-trigger-workflows#pull_request

@dvkruchinin
Copy link
Contributor Author

It should be borne in mind that changing the title of the PR or transferring it to the status "ready for review" does not restart jobs.

@dvkruchinin Hi, is it possible to use action types like edited or ready_for_review to get around this limitation?https://docs.github.com/en/actions/learn-github-actions/events-that-trigger-workflows#pull_request

Hi @azhavoro,
Yes. It`s works. Thanks. If necessary, I can add more types in the future.

@dvkruchinin
Copy link
Contributor Author

Specifying types overrides the list of types and does not complement it. Added the default ones.

@nmanovic nmanovic merged commit e3eeccb into cvat-ai:develop Oct 18, 2021
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