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

support content path update #4663

Draft
wants to merge 46 commits into
base: master
Choose a base branch
from

Conversation

barryyosi-panw
Copy link
Contributor

@barryyosi-panw barryyosi-panw commented Nov 12, 2024

Related Issues

fixes: CIAC-12161

Description

Within the content contributions-flow, the ValidateContent script uses demisto-sdk as a library.
The script constructs an isolated, local temporary Content repository which its path is required when using demisto-sdk's relevant functionality.

Currently the CONTENT_PATH constant is resolved statically (for obvious reasons) when importing the required modules, which prevents the script from utilizing it for validation and lint of the local temporary content repo.

These changes expose an updating functionality for the global CONTENT_PATH constant and all of its derivations.

Copy link

Changelog(s) in markdown:

  • Introducing functionality for updating CONTENT_PATH global. #4663

Copy link

Changelog(s) in markdown:

  • Introducing functionality for updating CONTENT_PATH global. #4663

@barryyosi-panw barryyosi-panw force-pushed the support-content-path-dynamic-update branch from 456fd89 to 2978aff Compare November 17, 2024 10:01
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This fixes the change introduced here #4657

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.

1 participant