Skip to content

add setting to never offer ignoring default branch pr #5435

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

Merged
merged 1 commit into from
Nov 6, 2023

Conversation

kurowski
Copy link
Contributor

@kurowski kurowski commented Nov 3, 2023

In my organization's workflow, we always have a PR associated with the default branch. And because we're heavy users of Dev Containers and Codespaces, our vscode workspaces are fairly ephemeral so we're always creating new ones.

Which means that in new workspaces we always see "there's a pull request associated with the default branch, do you want to ignore" notification and have to click the "Don't show" button. And since we create new environments frequently we see this notification frequently.

This pull request adds a setting to disable the offer to ignore PRs on the default branch, which improves our team's developer experience.

@kurowski
Copy link
Contributor Author

kurowski commented Nov 4, 2023

@microsoft-github-policy-service agree company="University of California Education Abroad Program"

@alexr00
Copy link
Member

alexr00 commented Nov 6, 2023

@kurowski is there a reason you can't use the existing setting githubPullRequests.ignoredPullRequestBranches?

@kurowski
Copy link
Contributor Author

kurowski commented Nov 6, 2023

@alexr00 I believe that would work if we always wanted to ignore the pull request on the default branch, but the problem is that we never want to ignore the pull request. I can't figure out a way to get that behavior from the available settings.

@alexr00
Copy link
Member

alexr00 commented Nov 6, 2023

I misunderstood, thanks for clarifying!

@alexr00 alexr00 added this to the November 2023 milestone Nov 6, 2023
Copy link
Member

@alexr00 alexr00 left a comment

Choose a reason for hiding this comment

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

Looks good, thanks for the PR!

@alexr00 alexr00 enabled auto-merge (squash) November 6, 2023 15:33
@alexr00 alexr00 self-assigned this Nov 6, 2023
@alexr00 alexr00 merged commit 42ef50a into microsoft:main Nov 6, 2023
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