Page MenuHomeFreeBSD
Differential D484

Update venture37 consultancy entry
ClosedPublic
Actions

Authored by allanjude on Jul 25 2014, 2:38 PM.

Details

Reviewers
eadler
wblock
bcr
Summary

PR: 191407
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=191407

Diff Detail

Repository
rD FreeBSD doc repository - subversion
Lint
No Linters Available
Unit
No Unit Test Coverage

Event Timeline

allanjude updated this revision to Diff 836.Jul 25 2014, 2:38 PM
allanjude retitled this revision from to Update venture37 consultancy entry.
allanjude updated this object.
allanjude edited the test plan for this revision. (Show Details)
allanjude added reviewers: bcr, wblock, eadler.
bcr edited edge metadata.Jul 25 2014, 6:09 PM

My 2 cents

share/xml/commercial.consult.xml
2207

Full stop after the Ltd abbreviation? Nor sure, maybe not in titles. Perhaps we have other entries in there already. Keeping this consistent might be another can of worms, though. Warren might know...

2210

shouldn't it be Open Source (capital S)?

wblock added inline comments.Jul 25 2014, 9:14 PM
share/xml/commercial.consult.xml
2207

I'd say this depends on the local conventions.

2210

Both words should be capitalized the same, whether it's upper or lower case.

2210–2214

s/&/and/

allanjude updated this revision to Diff 1377.Sep 6 2014, 5:29 PM
allanjude edited edge metadata.

Fix issues pointed out by bcr@ and wblock@

allanjude updated this revision to Diff 1378.Sep 6 2014, 5:31 PM

Fix oxford comma

bcr accepted this revision.Sep 6 2014, 7:18 PM
bcr edited edge metadata.
This revision is now accepted and ready to land.Sep 6 2014, 7:18 PM
allanjude closed this revision.Sep 6 2014, 7:47 PM

Commited as rD45556

share/xml/commercial.consult.xml
2210

doceng should make a decision about 'open source' vs 'Open Source' etc, and add it to the FDP wordlist

Revision Contents
Changeset List

PathSize
share/
xml/
14 lines

Diff 1378

View Options

share/xml/commercial.consult.xml

Loading...