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

Introduce content paths update functionallity #4677

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

Conversation

barryyosi-panw
Copy link
Contributor

@barryyosi-panw barryyosi-panw commented Nov 24, 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.

@barryyosi-panw barryyosi-panw force-pushed the support-content-path-dynamic-update-2 branch from bda8a6a to f8125f3 Compare November 25, 2024 08:23
@barryyosi-panw barryyosi-panw changed the title fix logger initialization Introduce content paths update functionallity Nov 25, 2024
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