Need to have proper permissions settings.
Nobody has claimed this yet.
- Dominant language
- MDX
- Stars
- 98
- Forks
- 298
- Avg merge
- 14h 38m
- Merged PRs (30d)
- 38
Description
cf https://github.com/nf-core/dualrnaseq/pull/3
This PR was merge (there was actually enough agreements, but I still think it as been merged) by someone who is on slack but not on the channel for this pipeline and who I haven't seen any comment on the PR in question either.
Not sure of the current involvement in the project.
From what I understood, this issue has happened already with the kmermaid pipeline too.
So I really think that stricter permissions settings should be enforced especially for merging PR.
I'm tagging the @nf-core/core team for now, but any idea is welcome.
Is your feature request related to a problem? Please describe.
A clear and concise description of what the problem is. Ex. I'm always frustrated when [...]
Describe the solution you'd like
A clear and concise description of what you want to happen.
Describe alternatives you've considered
A clear and concise description of any alternative solutions or features you've considered.
Additional context
Add any other context or screenshots about the feature request here.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by reviewing the linked nf-core/dualrnaseq pull request and the issue's reference to the kmermaid pipeline. Identify the current permission and merge settings, then define and document stricter enforcement with the nf-core/core team; done means the agreed permissions prevent unauthorized merges.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- github
- Domain
- authorization
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 20/100