Skip to content

README: E.123 number for meeting call - #748

Merged
vbatts merged 1 commit into
opencontainers:masterfrom
wking:meeting-copy-edit
Apr 5, 2017
Merged

README: E.123 number for meeting call#748
vbatts merged 1 commit into
opencontainers:masterfrom
wking:meeting-copy-edit

Conversation

@wking

@wkingwking commented Mar 29, 2017

Copy link
Copy Markdown
Contributor

And fix a period/paren swap. The period/paren swap and contry-code addition both bring the local version more in line with the image-spec version and seem like good changes.

I've diverged from the image-spec version by using space separation in the phone number instead of hyphens. From E.123:

9.1 Grouping of digits in a telephone number⁵ should be accomplished by means or spaces⁶ unless an agreed upon explicit symbol (e.g. hyphen) is necessary for procedural purposes. Only spaces should be used in an international number.

And fix a period/paren swap. The period/paren swap and contry-code
addition both bring the local version more in line with the image-spec
version [1] and seem like good changes.
I've diverged from the image-spec version by using space separation in
the phone number instead of hyphens. From E.123 [2,3]:
9.1 Grouping of digits in a telephone number [5] should be
accomplished by means or spaces [6] unless an agreed upon explicit
symbol (e.g. hyphen) is necessary for procedural purposes. Only
spaces should be used in an international number.
[1]: https://github.com/opencontainers/image-spec/blob/v1.0.0-rc5/README.md#weekly-call
[2]: http://www.itu.int/rec/T-REC-E.123/en
[3]: https://www.itu.int/rec/dologin_pub.asp?lang=e&id=T-REC-E.123-200102-I!!PDF-E&type=items
Signed-off-by: W. Trevor King <wking@tremily.us>
@hqhq

hqhq commented Apr 5, 2017

Copy link
Copy Markdown
Contributor

LGTM

Approved with PullApprove

1 similar comment
@vbatts

vbatts commented Apr 5, 2017

Copy link
Copy Markdown
Member

LGTM

Approved with PullApprove

@vbatts
vbatts merged commit e8a0d3d into opencontainers:masterApr 5, 2017
@wking
wking deleted the meeting-copy-edit branch April 5, 2017 16:20
@vbattsvbatts mentioned this pull request Jul 5, 2017
Sign up for freeto 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.

3 participants

@wking@hqhq@vbatts