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.

Sign up for GitHub

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

Jump to bottom

Add tags and exclude-tags support #2286

Merged
merged 9 commits into from
Aug 5, 2024
Merged

Add tags and exclude-tags support #2286

merged 9 commits into from
Aug 5, 2024

Conversation

pdenert
Copy link
Collaborator

@pdenert pdenert commented Aug 1, 2024

fix #492

@github-actions github-actions bot added package: patrol Related to the patrol package (native automation, test bundling) package: patrol_cli Related to the patrol_cli package labels Aug 1, 2024
@pdenert
Add exclude-tags
287abaa 
Use BooleanSelector
@pdenert pdenert changed the title Add tags support Add tags and exclude-tags support Aug 2, 2024
Base automatically changed from add-native-skip to master August 2, 2024 13:39
mateuszwojtczak
mateuszwojtczak approved these changes Aug 2, 2024
View reviewed changes
Copy link
Contributor

@mateuszwojtczak mateuszwojtczak left a comment

Choose a reason for hiding this comment

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

LGTM

packages/patrol/lib/src/common.dart Show resolved Hide resolved
@github-actions github-actions bot added the docs Concerns doc comments, READMEs, or docs label Aug 2, 2024
piotruela
piotruela approved these changes Aug 5, 2024
View reviewed changes
Copy link
Collaborator

@piotruela piotruela left a comment

Choose a reason for hiding this comment

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

Nice 👌

@pdenert pdenert merged commit 4727aad into master Aug 5, 2024
20 of 27 checks passed
@pdenert pdenert deleted the add-tags-from-skip-branch branch August 5, 2024 11:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mateuszwojtczak mateuszwojtczak mateuszwojtczak approved these changes

@piotruela piotruela piotruela approved these changes

Assignees
No one assigned
Labels
docs Concerns doc comments, READMEs, or docs package: patrol_cli Related to the patrol_cli package package: patrol Related to the patrol package (native automation, test bundling)
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add support for test tags to the CLI
3 participants
@pdenert @piotruela @mateuszwojtczak