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

[demisto-sdk] Add skipped paths to old_validate_manager #4309

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

[demisto-sdk] Add skipped paths to old_validate_manager #4309

wants to merge 2 commits into from

Conversation

samuelFain
Copy link
Contributor

@samuelFain samuelFain commented May 28, 2024 •
edited
Loading

Related Issues

relates: CIAC-10490

Description

Add SKIPPED_PATHS constant, which is used to exclude specific paths when using --include-untracked flag when running demisto-sdk validate.

@samuelFain samuelFain self-assigned this May 28, 2024
@samuelFain samuelFain requested a review from YuvHayun May 28, 2024 14:40
@github-actions GitHub Actions
Copy link

Changelog(s) in markdown:

  • Added exclusion for certain file paths when using validate with a --include-untracked flag. #4309

dorschw
dorschw requested changes May 30, 2024
View reviewed changes
Copy link
Contributor

@dorschw dorschw left a comment

Choose a reason for hiding this comment

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

offline discussion

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@dorschw dorschw dorschw requested changes

@YuvHayun YuvHayun Awaiting requested review from YuvHayun

Requested changes must be addressed to merge this pull request.

Assignees

@samuelFain samuelFain

Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@samuelFain @dorschw