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

Update README.md #64

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Update README.md #64

wants to merge 2 commits into from

Conversation

LaQuay
Copy link

@LaQuay LaQuay commented Jun 2, 2023

Update readme for being consistent in startServer / stopServer calls. What the Readme currently states is OK with the start and stop definitions, but it is not consistent.

If a developer tries copying/pasting now, it will fail.

With the proposed change, the server is started 'before' and 'after' execution.

update readme for being consistent in startServer / stopServer calls.
@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Jun 2, 2023

CLA Signed

The committers listed above are authorized under a signed CLA.

@LaQuay
Copy link
Author

LaQuay commented Jun 2, 2023

I also propose providing an example for the unload method.

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