Page MenuHomeFreeBSD

Start updating the Subversion primer.
Needs ReviewPublic

Authored by mat on May 30 2017, 4:42 PM.

Details

Summary

This kinda started by me trying to update the new committer's section...

Diff Detail

Lint
No Linters Available
Unit
No Unit Test Coverage
Build Status
Buildable 17441
Build 17273: arc lint + arc unit

Event Timeline

mat created this revision.May 30 2017, 4:42 PM
bcr added a subscriber: bcr.

Adding Michael Dexter for his work on D9981.

mat updated this revision to Diff 29074.May 31 2017, 1:28 PM
  • Update bits of the Subversion primer.
  • Move the properties things in a properties section.
eadler accepted this revision.Jun 18 2018, 2:30 AM
eadler added a subscriber: eadler.

LGTM.

wblock added a subscriber: wblock.Jun 18 2018, 12:15 PM
wblock added inline comments.
en_US.ISO8859-1/articles/committers-guide/article.xml
868–870

About the "edit it": that is not needed and kind of confuses the issue. A file does not have to be modified to be added to version control.

mat updated this revision to Diff 44064.Jun 19 2018, 11:10 AM
  • Address wblocks's comment.
mat updated this revision to Diff 44072.Jun 19 2018, 1:39 PM
  • Do not use svn abbreviated commands.
  • Always use svn delete
  • More fixes/changs.
bcr added a comment.Jun 19 2018, 2:24 PM

Just a formatting thing that I found, the rest looks fine.

en_US.ISO8859-1/articles/committers-guide/article.xml
1444

The indentation of the <important> tags is correct, here, but the <para> tags must now be at a deeper indentation level.

mat added inline comments.Jun 20 2018, 8:47 AM
en_US.ISO8859-1/articles/committers-guide/article.xml
1444

Well, obviously, but if I reindent everything properly, I then get told that I should not mix content change and whitespace changes :-)
So all whitespace changes are left for a later commit.