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.

Sign up for GitHub

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

Manage GitHub settings via Probot #654

Open
wants to merge 14 commits into
base: master
Choose a base branch
from
Open

Manage GitHub settings via Probot #654

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
14 commits
Select commit Hold shift + click to select a range
85cbed8
Manage GitHub settings via Probot
dhoppe May 12, 2020
2cee4c2
Migrate to .erb templates
dhoppe May 12, 2020
e141442
Omit configuration of required_pull_request_reviews, required_status_…
dhoppe May 13, 2020
ce9f1e3
Refactor settings.yml.erb in favor of to_yaml()
dhoppe May 13, 2020
d3a3171
Add required parameters
dhoppe May 13, 2020
b03dd16
Add gsub() because we need to keep the value null
dhoppe May 13, 2020
87b74d2
Make CODEOWNERS configurable
dhoppe May 13, 2020
aef8dea
Refactor CODEOWNERS.erb
dhoppe May 13, 2020
9c738c3
Disable notifications for @voxpupuli/collaborators
dhoppe May 14, 2020
e77e6f5
Prevent new line at the end of file
dhoppe May 14, 2020
be28ac3
Disable pull request reviews for master branch
dhoppe May 14, 2020
1336e19
Allow force pushes and deletions for modulesync branch
dhoppe May 14, 2020
0378109
Remove parameter delete_branch_on_merge, because it is not supported …
dhoppe May 15, 2020
78289bf
Do not force signed commits for master branch
dhoppe May 19, 2020
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
36 changes: 36 additions & 0 deletions config_defaults.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,37 @@
---
.github/settings.yml:
repository:
private: false
has_issues: true
has_projects: false
has_wiki: false
has_downloads: true
default_branch: master
Copy link
Contributor

Choose a reason for hiding this comment

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

Maybe out of scope for this specific PR, but has there been discussion of changing the default branch name to main?

Copy link
Member

Choose a reason for hiding this comment

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

AFAIK there has not been any discussion on this.

allow_squash_merge: true
allow_merge_commit: true
allow_rebase_merge: true
archived: false
branches:
- name: master
protection:
required_pull_request_reviews: undef
required_status_checks:
strict: true
contexts: []
enforce_admins: undef
restrictions: undef
required_signatures: false
- name: modulesync
protection:
required_pull_request_reviews: undef
required_status_checks:
strict: true
contexts: []
enforce_admins: undef
restrictions: undef
required_signatures: true
allow_force_pushes: true
allow_deletions: true
.travis.yml:
before_install:
- yes | gem update --system
Expand Down Expand Up @@ -35,6 +68,9 @@
channels:
- "chat.freenode.org#voxpupuli-notifications"
user: puppet
CODEOWNERS:
permissions:
'.github/settings.yml': '@voxpupuli/project-maintainers'
Gemfile:
required:
':test':
Expand Down
1 change: 1 addition & 0 deletions moduleroot/.github/settings.yml.erb
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
<%= @configs.slice('repository', 'labels', 'milestones', 'collaborators', 'teams', 'branches').to_yaml.gsub(/undef/, 'null') -%>
3 changes: 3 additions & 0 deletions moduleroot/CODEOWNERS.erb
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
<% @configs['permissions']&.each do |key, value| -%>
<%= key %> <%= value %>
<% end -%>