Differential D35071

Committers-guide port section: remove addport
ClosedPublic
Actions

Authored by pizzamig on Apr 26 2022, 6:37 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Dec 22, 1:46 PM

	Unknown Object (File)
	Sun, Dec 22, 1:42 PM

	Unknown Object (File)
	Sun, Dec 22, 1:20 PM

	Unknown Object (File)
	Sun, Dec 22, 1:18 PM

	Unknown Object (File)
	Thu, Nov 28, 10:54 PM

	Unknown Object (File)
	Nov 14 2024, 8:26 PM

	Unknown Object (File)
	Nov 14 2024, 12:19 PM

	Unknown Object (File)
	Nov 12 2024, 9:49 PM

Subscribers

pauamma_gundo.com

Details

Reviewers

tcberner
se
rene
pauamma_gundo.com
0mp

Group Reviewers

Commits

R9:e1f62cbb8da6: Committers-guide port section: remove addport

Summary

The script addport is not working anymore, it's based on SVN and
it's quite hard to port it to git.
Before its removal, we want to remove its references in the
documentation.
Related bugzilla entry:
https://bugs.freebsd.org/bugzilla/show_bug.cgi?id=261378

Diff Detail

Repository

R9 FreeBSD doc repository

Lint

No Lint Coverage

Unit

No Test Coverage

Build Status

Buildable 45479
Build 42367: arc lint + arc unit

Event Timeline

pizzamig requested review of this revision.Apr 26 2022, 6:37 PM

pizzamig created this revision.

Harbormaster completed remote builds in B45385: Diff 105445.Apr 26 2022, 6:37 PM

pizzamig added reviewers: rene, portmgr.Apr 26 2022, 6:39 PM

lgtm

rene accepted this revision.Apr 28 2022, 7:31 AM

This revision is now accepted and ready to land.Apr 28 2022, 7:31 AM

pauamma_gundo.com requested changes to this revision.Apr 28 2022, 7:28 PM

pauamma_gundo.com added a subscriber: pauamma_gundo.com.

pauamma_gundo.com added inline comments.

documentation/content/en/articles/committers-guide/_index.adoc
3686	https://webaim.org/techniques/hypertext/#alpha_links

This revision now requires changes to proceed.Apr 28 2022, 7:28 PM

See also https://reviews.freebsd.org/D35104 which removes addport itself.

Following up on the comment about screen readers

Harbormaster completed remote builds in B45461: Diff 105673.May 4 2022, 2:18 PM

Remove unwanted changes

Harbormaster completed remote builds in B45462: Diff 105674.May 4 2022, 2:21 PM

pizzamig marked an inline comment as done.May 4 2022, 2:21 PM

pauamma_gundo.com accepted this revision.May 5 2022, 2:34 PM

This revision is now accepted and ready to land.May 5 2022, 2:34 PM

My lack of confidence with arc removed the previous commit

This revision now requires review to proceed.May 5 2022, 7:45 PM

Harbormaster completed remote builds in B45479: Diff 105713.May 5 2022, 7:45 PM

Once those minor nits are addressed, it is ready to land.

Approved by: 0mp (doc)

PS Bonus points for avoiding "you" in the documentation. ;)

documentation/content/en/articles/committers-guide/_index.adoc
3670	Please keep one sentence per line. Use backticks around SUBDIR.
3686	Missing dot at the end.

This revision is now accepted and ready to land.May 5 2022, 7:55 PM

Addressing 0mp feedback

This revision now requires review to proceed.May 5 2022, 8:39 PM

Harbormaster completed remote builds in B45480: Diff 105714.May 5 2022, 8:40 PM

Put backticks in the right SUBDIR

Harbormaster completed remote builds in B45481: Diff 105715.May 5 2022, 8:44 PM

pizzamig marked an inline comment as done.May 5 2022, 8:45 PM

This revision was not accepted when it landed; it landed in state Needs Review.May 9 2022, 9:04 AM

Closed by commit R9:e1f62cbb8da6: Committers-guide port section: remove addport (authored by pizzamig). · Explain Why

This revision was automatically updated to reflect the committed changes.

pizzamig added a commit: R9:e1f62cbb8da6: Committers-guide port section: remove addport.

Revision Contents
Changeset List

Path

Size

documentation/

content/

en/

articles/

committers-guide/

24 lines

Diff 105713

documentation/content/en/articles/committers-guide/_index.adoc

Loading...