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

Enabled tests #991

Merged
merged 1 commit into from
Feb 14, 2024
Merged

Enabled tests #991

merged 1 commit into from
Feb 14, 2024

Conversation

amartya4256
Copy link
Contributor

@amartya4256 amartya4256 commented Dec 7, 2023

What it does

Enabled some disabled tests which aren't in broken state anymore.
eclipse-platform/eclipse.platform#525

How to test

Author checklist

@jukzi
Copy link
Contributor

jukzi commented Dec 8, 2023

@iloveeclipse i accidentally pressed update branch button which created a merge commit. can you disable that button? see https://github.com/orgs/community/discussions/3245#discussioncomment-7339067

@iloveeclipse
Copy link
Member

@iloveeclipse i accidentally pressed update branch button which created a merge commit. can you disable that button? see https://github.com/orgs/community/discussions/3245#discussioncomment-7339067

I do not see options shown on referenced screen.

All "merges" are disabled for this repo, yet "Update with merge" is still shown for PR's here. I believe that is exactly what the last comment on the thread youare referring to is saying.

@akurtakov
Copy link
Contributor

jukzi added a commit to jukzi/.eclipsefdn that referenced this pull request Dec 8, 2023
@netomi
Copy link

netomi commented Dec 8, 2023

I do believe this comes from the setting:

allow_update_branch: true

which will show a button in the PR if the PR branch can be updated with changes from the target branch prior to merging the PR.

The setting

allow_merge_commit: true

will affect the options that you get when you actually wanna merge the PR, but the merge commit happened before the merge to the target branch.

@jukzi
Copy link
Contributor

jukzi commented Dec 8, 2023

will allow_update_branch=false still show "rebase" button?

@netomi
Copy link

netomi commented Dec 9, 2023

No, it will not show any button anymore, so if you just wanna prevent merge commits from happening, the setting that you change will be good imho.

@jukzi
Copy link
Contributor

jukzi commented Feb 14, 2024

@amartya4256 tests are green now. thanks for your contribution.

@jukzi jukzi merged commit 5b4bdb4 into eclipse-jdt:master Feb 14, 2024
9 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.

5 participants