Skip to content

Move adeak from collaborators to developers#19

Closed
adeak wants to merge 1 commit into
pyvista:mainfrom
adeak:patch-1
Closed

Move adeak from collaborators to developers#19
adeak wants to merge 1 commit into
pyvista:mainfrom
adeak:patch-1

Conversation

@adeak

@adeak adeak commented Jun 19, 2026

Copy link
Copy Markdown
Member

I don't expect this to become a regular occurrence for the time being (unfortunately), but this week I reviewed two PRs and it's a bit cumbersome not to have write access (my approvals are currently homeopathic, and I can't even resolve review threads).

I'd like commit rights to make the little work I can do be more efficacious :) If in time the expected benefit of write permissions is again diminished compared to any security risk, we can kick me back out of the group.

cc @banesullivan

Signed-off-by: Andras Deak <adeak@users.noreply.github.com>
@adeak adeak requested a review from a team as a code owner June 19, 2026 19:35
@adeak

adeak commented Jun 19, 2026

Copy link
Copy Markdown
Member Author

Dry run is broken. Is the issue that I have to use a fork? https://github.com/pyvista/admin/actions/runs/27844960844/job/82412058365?pr=19
image

@banesullivan

Copy link
Copy Markdown
Member

Ah, yes. I have the this repo pretty locked down so it probably cannot run these checks from forks. I will merge this change. We may need to instead have issue templates for future requests like this instead so that a developer/admin can issue the PR internally.

@banesullivan

Copy link
Copy Markdown
Member

Replaced with #20

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

2 participants