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

Feature: Enhance the visibility of the "Update branch button" section #1527

Open
StanFromIreland opened this issue Mar 4, 2025 · 4 comments

Comments

@StanFromIreland
Copy link
Contributor

Describe the enhancement or feature you would like

This is something I find myself informing new contributors about daily. It should be made more visible in some way. (Then there is the problem of increasing the viability of the devguide itself...)

For now I think it would be good to move the section to just below the submitting section rather than the bottom of the file.

Describe alternatives you have considered

No response

Additional context

No response

@StanFromIreland StanFromIreland changed the title Feature: Enhance the visibility of the "Update branch button" Feature: Enhance the visibility of the "Update branch button" section Mar 4, 2025
@zware
Copy link
Member

zware commented Mar 4, 2025

I'm not clear on why this needs to move, or why it's being pointed out daily. The last point of the section is very relevant:

Do not click Update branch without a good reason because it notifies everyone watching the PR that there are new changes, when there are not, and it uses up limited CI resources.

In many cases, it's probably best that it not be used except by someone merging the PR.

@StanFromIreland
Copy link
Contributor Author

StanFromIreland commented Mar 6, 2025

It's being pointed out daily because new contributors are updating every time there is a new commit (e.g. the button appears)

I just want the section to be moved to a more visible place so that it is read!

@zware
Copy link
Member

zware commented Mar 6, 2025

Ahh, I see now!

I wonder if a better solution might be for @bedevere-bot to watch for the merge commits created by the button, and complain if they look abusive (more than one in a row, more than one in a day, multiple without a diff, etc.).

@StanFromIreland
Copy link
Contributor Author

I posted in the docs discord about welcome-bot which would post a message when someone opens their first pr? Maybe that would work too?

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

No branches or pull requests

2 participants