app: Branch protection not being applied
None of the repos where I have branch protection configured gets the settings applied.
I did two different tests without success.
- new repo without any rule, rules are not created
- existing repo with rule for
master, settings are not applied
Initially, I thought that this could be related to #136. That’s why I did test num 2.
Test were done on this repo: https://github.com/renatocaval/oh-my-git/ https://github.com/renatocaval/oh-my-git/blob/master/.github/settings.yml#L13-L23
About this issue
- Original URL
- State: open
- Created 5 years ago
- Reactions: 3
- Comments: 19 (10 by maintainers)
Commits related to this issue
- Fix `master` branch protection See: https://github.com/probot/settings/issues/150#issuecomment-543353809 — committed to johnmartel/.github by johnmartel 4 years ago
- Fix `master` branch protection See: https://github.com/probot/settings/issues/150#issuecomment-543353809 — committed to johnmartel/.github by johnmartel 4 years ago
- Fix `master` branch protection See: https://github.com/probot/settings/issues/150#issuecomment-543353809 — committed to johnmartel/.github by johnmartel 4 years ago
- Fix `master` branch protection See: https://github.com/probot/settings/issues/150#issuecomment-543353809 — committed to johnmartel/.github by johnmartel 4 years ago
- Fix `master` branch protection See: https://github.com/probot/settings/issues/150#issuecomment-543353809 — committed to johnmartel/.github by johnmartel 4 years ago
- Fix `master` branch protection See: https://github.com/probot/settings/issues/150#issuecomment-543353809 — committed to johnmartel/.github by johnmartel 4 years ago
- Fix `master` branch protection See: https://github.com/probot/settings/issues/150#issuecomment-543353809 — committed to johnmartel/.github by johnmartel 4 years ago
- Fix `master` branch protection See: https://github.com/probot/settings/issues/150#issuecomment-543353809 — committed to johnmartel/.github by johnmartel 4 years ago
- Fix `master` branch protection See: https://github.com/probot/settings/issues/150#issuecomment-543353809 — committed to johnmartel/.github by johnmartel 4 years ago
- Fix `master` branch protection See: https://github.com/probot/settings/issues/150#issuecomment-543353809 — committed to johnmartel/.github by johnmartel 4 years ago
- Fix `master` branch protection See: https://github.com/probot/settings/issues/150#issuecomment-543353809 — committed to johnmartel/.github by johnmartel 4 years ago
- Fix `master` branch protection See: https://github.com/probot/settings/issues/150#issuecomment-543353809 — committed to johnmartel/.github by johnmartel 4 years ago
- Fix `master` branch protection See: https://github.com/probot/settings/issues/150#issuecomment-543353809 — committed to johnmartel/.github by johnmartel 4 years ago
- Update settings.yml Looking at https://github.com/probot/settings/issues/150 — committed to cucumber/multi_test by mattwynne 2 years ago
I can’t get this to work either. I’d love some visibility of what’s happening when it tries to apply my yaml!
You are right. It works if all top-level settings are included.
Thanks for the tip.
That’s my current one:
I won’t work if I set then to
falselike in:we should probably hide these details if true, but it looks like i had to set some properties that i wasn’t trying to configure to
nullin order to get my config to apply.could you try some variations of that to see if gets you working? if that does end up working, i would be interested in the combination that worked as well as those that didnt so that we can try to improve configuring these.