Skip to content

Conversation

@glemaitre
Copy link
Member

I think that we discussed in the past that:

  • we don't really need to assign someone to its own PR
  • we don't really point out "user-reported" because it does not really help

Removing those two things can reduce a bit the noise on the repository.

Copy link
Contributor

@auguste-probabl auguste-probabl left a comment

Choose a reason for hiding this comment

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

Works for me, thanks.

Copy link
Collaborator

@thomass-dev thomass-dev left a comment

Choose a reason for hiding this comment

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

What is the issue behind we don't really need to assign someone to its own PR ?
I find quite useful to see who is responsible (the face of the profil) for the PR at a glance.

image

@glemaitre
Copy link
Member Author

I find quite useful to see who is responsible (the face of the profil) for the PR at a glance.

It is redundant with the "by xxx" that you have just under the PR title.

I would reserve this assign when the author is not the person opening the PR because it get my attention then when doing the review (I need to address someone else).

Copy link
Collaborator

@thomass-dev thomass-dev left a comment

Choose a reason for hiding this comment

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

It is redundant with the "by xxx" that you have just under the PR title.

"by xxx" is not the face ^^.

I would reserve this assign when the author is not the person opening the PR because it get my attention then when doing the review (I need to address someone else).

In this case, you can always change the assignee by hand.

I don't see the noise for being assigned automatically when you are the author: whatever happens in the PR, you will be notified as author.

But let's try without assignee!

@thomass-dev thomass-dev merged commit edef92a into probabl-ai:main Oct 27, 2025
16 checks passed
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.

3 participants