Jump to content

Recommended Posts

Is there a way to only receive notifications for differentials I am subscribed to, e.g. as the author, added as reviewer, mentioned etc?

It is very hard to go through the relevant patches I want to check. 2 or 3 days and I have 200 unread notifications

Link to post
Share on other sites

I don't understand. When you click on the link above, you get to this page, right?

image.png

 

As written there, your notifications should already be limited to what you are subscribed to.

Do you have an example of a change that you would not have wanted to receive a notification for?

Link to post
Share on other sites

You're not subscribed to this diff, you shouldn't receive any notification about it. Are you talking about the public Activity stream? You cannot possibly catch up with everything. You should rather manually subscribe to what you are interested in, and monitor your emails or notifications depending on your preferences (by notifications I mean the small bell icon on the top left).

30 minutes ago, badosu said:

Interestingly I see this on the bottom of the page, but I didn't perform any action:

This is a bit confusing indeed. It is a summary of the pending actions you would perform if you clicked "Submit". By default it would subscribe you. If you write inline comments, or choose actions in the Add Action.. menu, they will be added to the summary, waiting for you to press Submit.

Link to post
Share on other sites

Ah! When seeing your last message I thought you had discovered a bug.

But I just discovered that Stan has added you to the list of owners of the Balancing changes. You are basically an automatic reviewer for all those changes.

@Stan` you probably shouldn't do that before explaining to people that it will flood their notifications/inbox :P

Link to post
Share on other sites

Part of the problem is that the “O11: Templates (Balancing)” group is automatically added to any patch that includes a file from the simulation/data/ or simulation/templates/. Ideally only those that are explicitly labelled [gameplay] would be. Just a few examples that don't affect gameplay balance at all but have the O11 group added nonetheless:

There are dozens more.

Link to post
Share on other sites
23 hours ago, Nescio said:

Part of the problem is that the “O11: Templates (Balancing)” group is automatically added to any patch that includes a file from the simulation/data/ or simulation/templates/. Ideally only those that are explicitly labelled [gameplay] would be. Just a few examples that don't affect gameplay balance at all but have the O11 group added nonetheless:

There are dozens more.

In this case the best thing to do would be to create a Herald rule which is triggered when a diff title contains [gameplay]. Anyone can create such a rule, to be notified whenever they want :)

The so-called "Code owners" (O11 and the like) are only determined by paths of modified files. I think it's still interesting to have that.

Link to post
Share on other sites
14 minutes ago, Itms said:

In this case the best thing to do would be to create a Herald rule which is triggered when a diff title contains [gameplay]. Anyone can create such a rule, to be notified whenever they want :)

The so-called "Code owners" (O11 and the like) are only determined by paths of modified files. I think it's still interesting to have that.

While I'm not opposed to the existence of code owners groups, the people currently included in O11 (@badosu, @borg-, @Feldfeld, @scythetwirler, @Stockfish, @ValihrAnt) are not interested in every patch, only in the subset that affect gameplay balance. Basically split the group in two:

  • “simulation data and templates” (current repository paths)
  • “gameplay balancing” (current people)

and add a global herald rule to set the gameplay balancing group as a reviewer to any patch that includes [gameplay] in its title.

  • Like 1
Link to post
Share on other sites

Sounds good. I suggest not touching the O11 code ownership. Code owners are sufficiently complicated as they are. I think we should keep restricting code owners to team members @Stan`

I will create instead a Balancing user group with the Herald rule Nescio proposes (y)

Link to post
Share on other sites
46 minutes ago, Itms said:

I will create instead a Balancing user group with the Herald rule Nescio proposes (y)

Thank you, that's better! @scythetwirler appears quite inactive and could be removed from O11 too.

One more thing, while I tend to put [gameplay] in the patch title, there is no guarantee others do exactly the same, so the https://code.wildfiregames.com/H13 won't capture e.g. GAMEPLAY: or [balance]. False positives (e.g. D368) are less annoying than false negatives (e.g. D2737), therefore maybe change the H13 conditions to any patch of which the tag or title includes:

  • balanc
  • Balanc
  • BALANC
  • gameplay
  • Gameplay
  • GAMEPLAY

i.e. no brackets or colons, and no -e or -ing.

Link to post
Share on other sites
38 minutes ago, Itms said:

I think mistakes or inconsistencies can be easily fixed by just adding Balancing as reviewers manually.

True, but the proposed six will catch a lot more than the current four.

Link to post
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

×
×
  • Create New...