Skip to content

Updates after feedback#1

Open
ta4ilka69 wants to merge 7 commits into
mainfrom
test-after-fixes
Open

Updates after feedback#1
ta4ilka69 wants to merge 7 commits into
mainfrom
test-after-fixes

Conversation

@ta4ilka69
Copy link
Copy Markdown
Owner

No description provided.

Comment thread README.md
- `csv_prefix`: CSV filename prefix (default `locust_stats`).
- `locust_file`: Path to your `locustfile.py` (optional; Locust auto-discovers if omitted).
- `args`: Raw CLI arguments appended to the Locust command (preferred way to configure Locust).
- `config_file`: Optional path to Locust config file, passed with `--config`.
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Does this need to be its own parameter or can it be passed in args?

from locust import HttpUser, task, between


_ids_lock = Lock()
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I dont have a problem with including an example locustfile, but its probably easier for people to understand if there isn't stuff in here that doesnt actually relate to the github action.

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