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

Add related file path to several validation errors #4774

Merged

Conversation

YuvHayun
Copy link
Contributor

Related Issues

fixes: CIAC-11389

Description

Copy link

Changelog(s) in markdown:

  • Fixed an issue where BA102 (Invalid content items) would mistakenly fail for integration and script nested folders. #4774
  • Fixed an issue where some validations would fail with the wrong file path in the error message. #4774
  • Removed DS106 "is valid description name" since it was duplicated by DS104 "no description file" from the new validate. #4774

Copy link
Contributor

@JudahSchwartz JudahSchwartz left a comment

Choose a reason for hiding this comment

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

amazing!!!

demisto_sdk/commands/validate/README.md Outdated Show resolved Hide resolved
@YuvHayun YuvHayun merged commit b457274 into master Feb 9, 2025
25 checks passed
@YuvHayun YuvHayun deleted the ensure_validate_errors_are_thrown_for_right_content_item branch February 9, 2025 13:30
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