Page MenuHomeFreeBSD
Differential D10993

Start updating the Subversion primer.
Needs ReviewPublic
Actions

Authored by mat on May 30 2017, 4:42 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Jun 6, 1:23 PM
Unknown Object (File)
Sat, Jun 1, 2:41 PM
Unknown Object (File)
Tue, May 28, 7:16 PM
Unknown Object (File)
May 15 2024, 1:14 AM
Unknown Object (File)
Apr 28 2024, 8:15 AM
Unknown Object (File)
Apr 27 2024, 3:27 PM
Unknown Object (File)
Apr 19 2024, 10:42 PM
Unknown Object (File)
Mar 11 2024, 3:30 AM
View All 32 Files
Subscribers
bcr
eadler
wblock
ygy

Details

Reviewers
editor_callfortesting.org
eadler
Group Reviewers
Doc Committers
Summary

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

Diff Detail

Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 9578
Build 10025: arc lint + arc unit

Event Timeline

mat created this revision.May 30 2017, 4:42 PM
Harbormaster completed remote builds in B9578: Diff 29025.May 30 2017, 4:42 PM
bcr added a reviewer: editor_callfortesting.org.May 30 2017, 4:47 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.
Harbormaster completed remote builds in B9602: Diff 29074.May 31 2017, 1:29 PM
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
942

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.
Harbormaster completed remote builds in B17438: Diff 44064.Jun 19 2018, 11:10 AM
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.
Harbormaster completed remote builds in B17441: Diff 44072.Jun 19 2018, 1:39 PM
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
1523

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
1523

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.

ygy added a subscriber: ygy.Dec 10 2020, 7:09 AM
ygy added a comment.Jun 12 2021, 5:35 AM

Is this still relevant?

Revision Contents
Changeset List

PathSize
en_US.ISO8859-1/
articles/
committers-guide/
112 lines

Diff 29025

View Options

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

Loading...