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 #3940

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Update README.md #3940

wants to merge 1 commit into from

Conversation

liamsegura
Copy link

Documentation update, fixed punctuation.

- What I did

Fixed documentation

- How I did it

with incredible difficulty

- How to verify it

eyes

- Description for the changelog

Huge documentation update

- A picture of a cute animal (not mandatory but encouraged)

Documentation update, fixed punctuation.
@deitch
Copy link
Collaborator

deitch commented Jul 11, 2023

Why is this needed?

@stappersg
Copy link

Why is this needed?

Some documentation conventions say each bullet point ends with a point (as a regular sentence)

Some documentation conventions say each bullet point is just a few words (and don't need a point at end)

This comment is however a plea for make a decission so we can move on. In stead of letting this open issue drain more human energy.

@deitch
Copy link
Collaborator

deitch commented Oct 30, 2023

Sure, this is fine, then. But it does need the signing and DCO.

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.

None yet

3 participants