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

fixed some technical terms in the readme #50

Merged
merged 2 commits into from
May 25, 2022
Merged

fixed some technical terms in the readme #50

merged 2 commits into from
May 25, 2022

Conversation

tropxy
Copy link
Contributor

@tropxy tropxy commented May 24, 2022

No description provided.

Copy link
Contributor

@sabrinagordon-believ sabrinagordon-believ left a comment

Choose a reason for hiding this comment

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

I was wondering about a couple of these myself.

@@ -17,8 +17,8 @@ There are two recommended ways of running the project:

### Running with Docker
Using Docker has the advantage of starting everything up automatically,
including certificate generation, tests and linting, as well as creating
both the SECC and EVCC.
including certificate generation, tests and linting, as well as spawning
Copy link
Contributor

Choose a reason for hiding this comment

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

Either is fine in this context, I think -- @nikkigithub1 was there a specific reason to change "spawning" to "creating" in the first place?

README.md Outdated Show resolved Hide resolved
@@ -192,21 +193,21 @@ The following table provides a few of the available variables:
| MESSAGE_LOG_EXI | `False` | Whether or not to log the EXI Bytestream messages (only works if log level is set to DEBUG)


## Licence
## License
Copy link
Contributor

Choose a reason for hiding this comment

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

I think this is fine -- American English is often the standard in a technical context.

@tropxy tropxy merged commit 59b8360 into master May 25, 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