[Eclipse Theia] [GitHub] Enable "Require branches to be up to date before merging" option

Summary

Hi,

Can you enable the "Require branches to be up to date before merging" option for our main repo?

repo: https://github.com/eclipse-theia/theia

The option can be found in the project's settings, under the "branches" side menu.

Thanks! Marc

Steps to reproduce

N/A

What is the current bug behavior?

N/A

What is the expected correct behavior?

N/A

Relevant logs and/or screenshots

Priority

• Urgent
• High
• Medium
• Low

Severity

• Blocker
• Major
• Normal
• Low

Impact

N/A

added IT-prioritylow IT-severitylow ~16280 statewip labels

assigned to @pstankie

I enabled the setting for default master and che-7.0.0 branches. I haven't enabled for codeowner_test.

Also, there's no check defined for master branch:

Please confirm this is what you wanted.

Also, there's no check defined for master branch:

@pstankie Thanks for letting me know - I expected that the checks we have defined in the repo's workflows would be the ones used for this, but apparently not. I will investigate and get back to you.

@pstankie can you set focus to the "Search for status checks [...]" textbox and see if proposes some of the checks we have defined in the repo? I would expect the following ones:

@pstankie I think I see what's going-on. Please hold-off for now - I think we will want to rename some of the "checks" before you add them to the list.

OK. I was adding a comment about names being dynamic and generating great number of them. I will wait.

@pstankie could you temporarily remove all check from the required list? I'm working on a PR to rename them so they will easier to recognize and eventually maintain. But the original names are marked as "required", which will block merging:

Removed.

@pstankie My PR is merged- thanks for your patience. I have discussed with another committer, and we think we should have the following checks, selected as being required:

Lint, eclipsefdn/eca

I think this may be what you already setup for us? If so, we are done here!

Yes, these are the checks that are enabled for master branch.

Closing.

closed

removed statewip label

mentioned in issue #3126 (closed)