get additional github administration rights for bugtracker

Erkan-Yilmaz / Gridcoin-tasks

tasks, wishes, ideas, ... for the Gridcoin project

26 stars 1 forks source link

get additional github administration rights for bugtracker #41

Closed Erkan-Yilmaz closed 7 years ago

Erkan-Yilmaz commented 7 years ago

so easy pull requests (typos) could be merged into the code
more difficult PRs should still require a certain procedure:
- tested on these OS (windows, linux, ...) ?
- done these, agreed on, test cases ?
- and also test cases which are relevant for the changed code parts ?
- unit tests were added ?
- ...
who should get the additional flag ?

see @denravonska 's comment here

assigning to @grctest

grctest commented 7 years ago

It should be assigned to @gridcoin because he's the only one who can change this.

denravonska commented 7 years ago

Just to reiterate what I said/wanted to say at the hangout.

Push privileges require a good bit of trust, especially for these kind of applications. As an alternative we could have a couple of trusted developers who can take the first code reviews. When the PRs have passed those developers Rob can have a final look and do the merge. Eventually some of those developers could have gained enough trust to get push privileges but for now it's a good middle ground.

I also agree about using a development branch for continuous development and PRs instead of pushing everything into master.

grctest commented 7 years ago

https://cryptocointalk.com/topic/52186-gridcoin-community-development-process-idea/#entry232500

We've created a gridcoin-community organization, where we can explore improved github workflow.

What tools should we directly integrate into the org or repos?

grctest commented 7 years ago

Closed because we've created the gridcoin-community repo to gain these permissions. I'll create an additional issue regarding the integration of tools.