Page MenuHomeFreeBSD

Add/replace eduardo email to ports I maintain
ClosedPublic

Authored by eduardo on Feb 8 2021, 11:30 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Nov 16, 10:08 AM
Unknown Object (File)
Fri, Nov 15, 2:06 PM
Unknown Object (File)
Mon, Nov 11, 11:44 AM
Unknown Object (File)
Wed, Nov 6, 10:57 PM
Unknown Object (File)
Wed, Oct 30, 2:28 PM
Unknown Object (File)
Oct 15 2024, 3:05 AM
Unknown Object (File)
Oct 15 2024, 3:05 AM
Unknown Object (File)
Oct 14 2024, 4:50 PM
Subscribers

Details

Summary

Update email address in all ports that I am the maintainer

Approved by: dbaio, garga (mentors)
Differential Revision: https://reviews.freebsd.org/D28534

Diff Detail

Repository
rP FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

@eduardo please, write the full commit message in the summary, there is an example in D28477.

eduardo retitled this revision from Add/replace eduado email to ports I maintain to Add/replace eduardo email to ports I maintain.
garga requested changes to this revision.Feb 8 2021, 12:43 PM

It must follow templates format so it should have Approved by: followed by a TAB and then people that approved. Differential line should be Differential Revision: followed by a TAB and the general URL for this review, which is https://reviews.freebsd.org/D28534

This revision now requires changes to proceed.Feb 8 2021, 12:43 PM

A single Approved by line is enough. And Differential Revision line ls mandatory. It would look like this:

Approved by:		dbaio, garga (mentors)
Differential Revision:	https://reviews.freebsd.org/D28534

Invalid!

The value you have entered in "Summary" can not be parsed unambiguously
when rendered in a commit message. Edit the message so that keywords like
"Summary:" and "Test Plan:" do not appear at the beginning of lines. Parsed

keys: revisionID.

  • replace email to eduardo on my 26 ports that I'm maintaining.

Approved by: dbaio, garga (mentors)

Differential Revision: https://reviews.freebsd.org/D28534

garga (Renato Botelho) <phabric-noreply@freebsd.org> escreveu no dia
segunda, 8/02/2021 à(s) 12:44:

garga requested changes to this revision.
garga added a comment.
This revision now requires changes to proceed.

It must follow templates format so it should have `Approved by:`

followed by a TAB and then people that approved. Differential line should
be Differential Revision: followed by a TAB and the general URL for this
review, which is https://reviews.freebsd.org/D28534

REPOSITORY

rP FreeBSD ports repository

CHANGES SINCE LAST ACTION

https://reviews.freebsd.org/D28534/new/

REVISION DETAIL

https://reviews.freebsd.org/D28534

EMAIL PREFERENCES

https://reviews.freebsd.org/settings/panel/emailpreferences/

To: eduardo, garga, dbaio
Cc: mat

Invalid!

Add an extra space in the beginning of the line that it will work.

Update email address in all ports that I am the maintainer

Approved by:    dbaio, garga (mentors)
Differential Revision:  https://reviews.freebsd.org/D28534
This revision is now accepted and ready to land.Feb 9 2021, 11:10 AM