Skip to content
This repository was archived by the owner on Apr 22, 2023. It is now read-only.

Conversation

AlexKVal
Copy link

its -> it's -> it is
a HTTP -> we read 'H' as 'eich' -> an HTTP

its -> it's -> it is
a HTTP -> we read 'H' as 'eich' -> an HTTP

Choose a reason for hiding this comment

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

@joyent/node-coreteam Could a native English speaker please confirm the usage of "an" instead of "a" here? Thank you!

Copy link

Choose a reason for hiding this comment

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

Confirmed per BBC:

Take the eighth letter of the alphabet, pronounce it haitch [...]

So "an HTTP" should be similar to "an hour". :)

@misterdjules misterdjules added this to the 0.12.3 milestone Apr 1, 2015
@misterdjules misterdjules modified the milestones: 0.12.3, 0.12.4 May 14, 2015
@misterdjules misterdjules modified the milestones: 0.12.4, 0.12.5 May 25, 2015
@misterdjules misterdjules modified the milestones: 0.12.5, 0.12.6 Jun 22, 2015
@misterdjules misterdjules modified the milestones: 0.12.6, 0.12.7, 0.12.8 Jul 6, 2015
@jasnell
Copy link
Member

jasnell commented Aug 14, 2015

Will land this in a separate commit and PR in http://github.com/nodejs/node master

@jasnell
Copy link
Member

jasnell commented Aug 14, 2015

Closing here. New PR opened.

@jasnell jasnell closed this Aug 14, 2015
@AlexKVal AlexKVal deleted the patch-2 branch August 14, 2015 18:40
jasnell added a commit to jasnell/node that referenced this pull request Aug 26, 2015
jasnell added a commit to nodejs/node that referenced this pull request Aug 26, 2015
per: nodejs/node-v0.x-archive#14352

originally submitted by @AlexKVal

Reviewed By: Sakthipriyan Vairamani <[email protected]>
PR-URL: #2378
jasnell added a commit to nodejs/node that referenced this pull request Aug 26, 2015
per: nodejs/node-v0.x-archive#14352

originally submitted by @AlexKVal

Reviewed By: Sakthipriyan Vairamani <[email protected]>
PR-URL: #2378
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants