Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

add Joseph 'casz' Petersen #15

Merged
merged 1 commit into from
Aug 29, 2016
Merged

add Joseph 'casz' Petersen #15

merged 1 commit into from
Aug 29, 2016

Conversation

jetersen
Copy link
Contributor

as new maintainer of Accurev plugin I would like to be added 馃帀

@orrc
Copy link
Member

orrc commented Aug 28, 2016

casz is indeed the new maintainer:
https://botbot.me/freenode/jenkins/2016-08-27/?msg=72058192&page=4
https://github.com/jenkinsci/accurev-plugin/commits/master?author=casz

Assuming LDAP and GitHub usernames are the same, 馃憤 from me.

@daniel-beck: Do we have a good way of verifying the LDAP-GitHub association? Do we want to remove old maintainers from the list (this plugin was up for adoption until recently)?

@jetersen
Copy link
Contributor Author

jetersen commented Aug 28, 2016

@orrc username is casz, all around.
only reference I gave was:
https://groups.google.com/forum/#!msg/jenkinsci-dev/SdNcPm5q_vY/wQ0rYg8uCQAJ

edit:
Entered my github id on my jenkins.io account, so I guess that would be the place to verify?
Removed the plugin from adoption earlier today.

@daniel-beck
Copy link
Contributor

@batmat Did you get a response by the previous maintainer before making @Casz the maintainer? @rym002 was still around not even a year ago.

@batmat
Copy link
Contributor

batmat commented Aug 28, 2016

@daniel-beck Nope, I proceeded immediately after seeing the plugin was already marked as to-be-adopted. I would have asked the previous maintainer(s) for sure as usual if not.

@rym002
Copy link

rym002 commented Aug 28, 2016

I am OK with this.

@daniel-beck
Copy link
Contributor

@rym002 Thanks!

@daniel-beck daniel-beck merged commit 5d2d972 into jenkins-infra:master Aug 29, 2016
daniel-beck pushed a commit that referenced this pull request Feb 4, 2017
* Remove old permissions from Accurev plugin

As suggested in #15 (comment)
Should we remove the old once?

Also is `org/jvnet/hudson/plugins/accurev` still relevant?

* Remove org.jvnet.hudson.plugins.accurev
@jetersen jetersen deleted the add-joseph-accurev branch January 18, 2018 18:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants