Skip to content

Updated Readme#5822

Closed
H3RSKO wants to merge 4 commits into
curl:masterfrom
H3RSKO:H3RSKO-readme-typo
Closed

Updated Readme#5822
H3RSKO wants to merge 4 commits into
curl:masterfrom
H3RSKO:H3RSKO-readme-typo

Conversation

@H3RSKO

@H3RSKO H3RSKO commented Aug 16, 2020

Copy link
Copy Markdown
Contributor

Fixed typo on Readme:

Changed "web site" to "website"

Fixed typo on Readme:

Changed "web site" to "website"

@bagder bagder left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Agreed! According to wikipedia:

While "web site" was the original spelling (sometimes capitalized "Web site", since "Web" is a proper noun when referring to the World Wide Web), this variant has become rarely used, and "website" has become the standard spelling

@bagder

bagder commented Aug 16, 2020

Copy link
Copy Markdown
Member

How about doing a git grep -i "web site" and update several other references while you're at it?

@H3RSKO

H3RSKO commented Aug 16, 2020

Copy link
Copy Markdown
Contributor Author

Updated every 'web site' in repo to 'website'.

@bagder

bagder commented Aug 16, 2020

Copy link
Copy Markdown
Member

I think you missed a few:

$ git grep -li "web site" | wc -l
14

@H3RSKO

H3RSKO commented Aug 16, 2020

Copy link
Copy Markdown
Contributor Author

Sorry about that! I think I got them all now, although 'git grep -li "web site" | wc -l' is still showing 1.

@bagder

bagder commented Aug 16, 2020

Copy link
Copy Markdown
Member

Is that perhaps because you missed the uppercase "WEB SITE" in README ?

That was it!
@H3RSKO

H3RSKO commented Aug 16, 2020

Copy link
Copy Markdown
Contributor Author

That was it

@bagder

bagder commented Aug 16, 2020

Copy link
Copy Markdown
Member

Thanks!

@bagder bagder closed this in 3d22140 Aug 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants