Skip to content

Conversation

Phenix333
Copy link
Contributor

It's of #3830.

Copy link

vercel bot commented May 8, 2025

@Colin-de-Seroux is attempting to deploy a commit to the github readme stats Team on Vercel.

A member of the Team first needs to authorize it.

@github-actions github-actions bot added the documentation Improvements or additions to documentation. label May 8, 2025
Copy link
Collaborator

@qwerty541 qwerty541 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello @Colin-de-Seroux

Thank you for opening this pull request! I appreciate your efforts and interest in our project.

Recently I checked out the branch on my local machine and did some documentation review. I failed to push the changes to the branch despite I see the mark "Maintainers are allowed to edit this pull request.". Looks like that happens due to some additional configuration in your fork repository.

I would be happy to merge the pull request when I will be able to push my changes.

Also I want to clarify whether have you tested the fine-grained token with provide permissions or not.

@Phenix333
Copy link
Contributor Author

Hello @qwerty541,

I don't know about the fork.

Regarding the fine-grained token, I tested different combinations and the limitations (write's in the readme of my push) are listed in one of the GitHub documentation pages (I can't remember which one).

@qwerty541
Copy link
Collaborator

Hello @qwerty541,

I don't know about the fork.

Regarding the fine-grained token, I tested different combinations and the limitations (write's in the readme of my push) are listed in one of the GitHub documentation pages (I can't remember which one).

Hello @Phenix333

I think the problem is that your forked repository is under organization instead of personal repository. I don't want to waste the time on access permissions, so recently I opened and merged another pull request #4426 which includes my review changes. I have added a commit sign Co-authored-by: Colin-de-Seroux <[email protected]>, so you still get a contribution in your profile.

Thank you for your awesome contribution!

@qwerty541 qwerty541 closed this Sep 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants