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

Minor edits to the documentation #2140

Merged
merged 6 commits into from
Jul 26, 2022
Merged

Minor edits to the documentation #2140

merged 6 commits into from
Jul 26, 2022

Conversation

sosna
Copy link
Contributor

@sosna sosna commented Jul 25, 2022

Hey there,

First, thanks for your nice project :).

While reading the documentation, I found a few typos or missing punctuation, and thought you might want to receive a pull request for these.

Cheers,
Xavier

PS: I read the contributing.md, but it does not seem to apply to changes to the documentation?

@cyberw
Copy link
Collaborator

cyberw commented Jul 25, 2022

Nice! I almost couldnt believe we still had that many errors. Have a look at my comments and I’ll merge.

@sosna
Copy link
Contributor Author

sosna commented Jul 26, 2022

I almost couldnt believe we still had that many errors.

Well, as a user, I found the documentation to be clear and comprehensive :)

Have a look at my comments and I’ll merge.

I have made the requested changes but I wasn't entirely sure about the first one and so didn't resolve that conversation.

@sosna
Copy link
Contributor Author

sosna commented Jul 26, 2022

Done :)

@cyberw
Copy link
Collaborator

cyberw commented Jul 26, 2022

Thanks!!

@cyberw cyberw merged commit 62b486b into locustio:master Jul 26, 2022
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