Skip to content

Conversation

Res260
Copy link
Contributor

@Res260 Res260 commented Jul 30, 2024

Add a launcher to contentctl.py to allow easier debugging and launching with python -m contentctl.contentctl

pyth0n1c
pyth0n1c previously approved these changes Aug 9, 2024
Copy link
Contributor

@pyth0n1c pyth0n1c left a comment

Choose a reason for hiding this comment

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

Yes, this looks like an improvement for some types of testing/debugging without doing a poetry install or pip install :)

@pyth0n1c
Copy link
Contributor

pyth0n1c commented Aug 9, 2024

For awareness of anything else looking at the PR, this addition means that you can do things like

python -m contentctl.contentctl validate
python -m contentctl.contentctl test --post-test-behavior always_pause mode:all

which is convenient, because you do not need to poetry install / poetry shell or pip install contentctl to run the tool as long as you have the requirements met.

@pyth0n1c pyth0n1c changed the base branch from main to release_v4.2.0 August 9, 2024 15:01
@pyth0n1c pyth0n1c changed the base branch from release_v4.2.0 to main August 9, 2024 15:02
@pyth0n1c pyth0n1c dismissed their stale review August 9, 2024 15:02

The base branch was changed.

@pyth0n1c pyth0n1c merged commit 2820c29 into splunk:main Aug 9, 2024
@Res260 Res260 deleted the easylaunch branch August 9, 2024 15:37
@Res260
Copy link
Contributor Author

Res260 commented Aug 9, 2024

Yes and it also allows to put breakpoints in IDEs like pycharm and VS code (idk if there is another way but its usually how I do it haha)

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.

3 participants