Skip to content

GSIP 180

Jody Garnett edited this page Oct 1, 2019 · 9 revisions

GSIP 180 - Committer and SF admin cleanup and rules refresh

Overview

The GeoServer committer list is long and includes people that have not been active for a while. This proposal includes both actions to clean it up, and updates to rules for committers that would facilitate cleanup in the future.

Proposed By

Andrea Aime

Assigned to Release

This proposal is not related to a specific GeoServer version.

State

  • Under Discussion
  • In Progress
  • Completed
  • Rejected
  • Deferred

Motivation

The current committer list has 44 members, including several people that have not been active in the project in a long while. Also, achieving committer status goes through a vote but comes with no requirements on the committer, in a time when we need more participation in the project.

The SourceForge project admins have access to the file release process and can manage mailing lists. Similar to committers, we have a long list with many inactive members, that need a clean up.

Proposal

The suggestion is to remove commit access from all members that have been completely inactive in the last year, and also to include in the committer description page that commit access decays after one year of complete inactivity. This would allow a quick initial cleanup.

As a further requirement for committers, we would mandate subscription to the three following lists:

  • geoserver-devel, to participate in the activity of the project
  • geoserver-builds, to be notified of build failure and pro-actively fix anything related to own activity
  • geoserver-users, to help users with any issue related to code that the committer have been working on

Failure to subscribe to the lists would also cause commit access removal.

GitHub committer cleanup implementation

The list of people making any commit in the last year (either directly or by PR) can be obtained by using:

git shortlog -nsu --since "one year ago"  | cut -f2 | sort

Cleaning up for obvious duplicates the current result (as of September 22nd) is:

Aegman
alesauvage
Alessandro Parma
Alessio Fabiani
Alex van den Hoogen
Andrea Aime
Andrea Antonello
Andreas Schmitz
Artem Groznykh
Ben Caradoc-Davies
Brad Hards
chosenegbe
Chris Hodgson
Damiano Albani
Daniele Romagnoli
Devon Tucker
dollarklavs
Emanuele Tajariol
Fernando Mino
François Dodé
Frank Gasdorf
Gabriel Roldan
GCOGit
gioscarda
hisham waleed karam
Ian Turton
Imran
James Hughes
JFBorsodi
Jody Garnett
johngschulz
Jorge Gustavo Rocha
Kevin Smith
Lorenzo Pini
Lubomír Bucek
Luca Morandini
Marcus Sen
Mauro Bartolomeoli
Meine Toonen
Michi, der
Morgan Thompson
Niels Charlier
Nuno Oliveira
Peter Rushforth
Peter Smythe
Raymond Nijssen
R. Hoek
Simone Giannecchini
Steve Ikeoka
Taba90
Tiago Vieira
tlann
Torben Barsballe
vickdw
Víctor González

Intersecting with the existing committer list results in the following active committers in the last year:

Alessandro Parma
Alessio Fabiani
Andrea Aime
Ben Caradoc-Davies
Brad Hards
Chris Hodgson
Damiano Albani
Daniele Romagnoli
Devon Tucker
Emanuele Tajariol
Fernando Mino
Frank Gasdorf
Gabriel Roldan
Ian Turton
James Hughes
Jody Garnett
Kevin Smith
Mauro Bartolomeoli
Morgan Thompson
Niels Charlier
Nuno Oliveira
Peter Rushforth
Simone Giannecchini
Steve Ikeoka
Torben Barsballe
vickdw
Víctor González

Sourceforce admin cleanup implementation

The current list of SourceForge admins contains the following:

  • Mike Pumphrey (mpumphrey)
  • Arne Kepp (arneke)
  • Ian Turton (ianturton)
  • Brent Owens (sploreg)
  • Chris Holmes (cholmesny)
  • Andrea Aime (aaime)
  • Mark Leslie (markles55)
  • jive (jive)
  • Gabriel Raul Roldan (groldan)
  • Justin Deoliveira (jdeolive)
  • Kevin Michael Smith (hai-etlik)
  • Torben Barsballe (tbarsballe)
  • Ben Caradoc-Davies (bencd)
  • GeoTools Builder (gtbuilder)

The idea is to leave only those that have been active doing any management in the last year, that is:

  • Ian Turton (ianturton)
  • Andrea Aime (aaime)
  • jive (jive)
  • Kevin Michael Smith (hai-etlik)
  • Torben Barsballe (tbarsballe)
  • GeoTools Builder (gtbuilder) - required for build.geoserver.org access

See [GeoServer Source Forge Page)(https://sourceforge.net/projects/geoserver/)

Backwards Compatibility

No backwards compatibility issues.

Feedback

Voting

Project Steering Committee:

  • Alessio Fabiani: +1
  • Andrea Aime: +1
  • Ian Turton: +1
  • Jody Garnett: +1
  • Jukka Rahkonen: +1
  • Kevin Smith: +1
  • Simone Giannecchini: +1

Community:

  • Nuno Oliveira: +1
  • Gabriel Roldan: +1

Links

Will add links as need be.

Clone this wiki locally