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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

use alert formatting #202

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

use alert formatting #202

wants to merge 1 commit into from

Conversation

musvaage
Copy link

This was implemented in DeepSeek-V3.

@musvaage
Copy link
Author

musvaage commented Feb 1, 2025

@DeepSeekPH

copy/paste from a pull in DeepSeek-V3

Some PRs in the sidebar list GH users as Reviewers who are not a repo Collaborator.

It seems the Organization might benefit from assigning a Collaborator to review PRs.

Such Collaborator only would then be listed as approving an individual PR.

Were write authority also granted that Collaborator could close PRs or mark them as Spam.

@musvaage
Copy link
Author

musvaage commented Feb 2, 2025

Regarding my previous comment.

Settings > Access > Moderation options

Code review limits

Restrict users who are permitted to approve or request changes on pull requests in this repository.

☐ Limit to users explicitly granted read or higher access

When enabled, only users explicitly granted access to this repository will be able to submit pull
request reviews that "approve" or "request changes". All users able to submit comment pull request
reviews will continue to be able to do so.

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.

2 participants