Skip to content

Commit 3a8d838

Browse files
authored
Merge pull request #7 from ft-circleci-orbs/add-repo-settings
Add GitHub settings to repo
2 parents 6bfa8c2 + b6e7c08 commit 3a8d838

File tree

3 files changed

+62
-0
lines changed

3 files changed

+62
-0
lines changed

.github/CODEOWNERS

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
* @ft-circleci-orbs/code-management

.github/pull_request_template.md

Lines changed: 28 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,28 @@
1+
## Why?
2+
3+
- What's the main motivation of this work?
4+
- What prompted you to start looking at these changes?
5+
- Is it a new feature, bug fix, tech debt?
6+
- Are there any links to other PRs or documentation that would be helpful for the review to understand the context of this PR?
7+
8+
## What?
9+
10+
- Please be specific and try to describe your thought process.
11+
- State the obvious, since this might be the first time the reviewer is looking at the code.
12+
- Include screenshots of the change if relevant.
13+
- Remember to update the documentation if relevant.
14+
15+
## Anything in particular you'd like to highlight to reviewers?
16+
17+
Mention here sections of code which you would like reviewers to pay extra attention to, for example:
18+
19+
- _I would appreciate a second pair of eyes on the test_
20+
- _I am not quite sure how this bit works_
21+
- _Is there a better module I can use, or approach I can take to achieve X_
22+
23+
24+
## PR Checklist
25+
26+
- [ ] Unit tests have been developed to cover current functionality.
27+
- [ ] Logging has been implemented to cover both informational and error levels.
28+
- [ ] Appropriate error handling has been built out.

.github/settings.yml

Lines changed: 33 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,33 @@
1+
# These settings are synced to GitHub by https://probot.github.io/apps/settings/
2+
3+
repository:
4+
default_branch: main
5+
delete_branch_on_merge: true
6+
topics: code-management
7+
8+
teams:
9+
- name: code-management
10+
permission: admin
11+
12+
branches:
13+
- name: main
14+
protection:
15+
required_pull_request_reviews: # Required. Require at least one approving review on a pull request, before merging. Set to null to disable.
16+
required_approving_review_count: 1 # The number of approvals required. (1-6)
17+
dismiss_stale_reviews: false # Dismiss approved reviews automatically when a new commit is pushed.
18+
require_code_owner_reviews: true # Blocks merge until code owners have reviewed.
19+
dismissal_restrictions: # Specify which users and teams can dismiss pull request reviews. Pass an empty dismissal_restrictions object to disable. User and team dismissal_restrictions are only available for organization-owned repositories. Omit this parameter for personal repositories.
20+
users: []
21+
teams: []
22+
23+
required_status_checks: # Required. Require status checks to pass before merging. Set to null to disable
24+
strict: true # Required. Require branches to be up to date before merging.
25+
contexts: [] # Required. The list of status checks to require in order to merge into this branch
26+
# checks: [] # Don't list any checks that must pass by default.
27+
28+
enforce_admins: true # Required. Enforce all configured restrictions for administrators. Set to true to enforce required status checks for repository administrators. Set to null to disable.
29+
restrictions: null # Required. Restrict who can push to this branch. Team and user restrictions are only available for organization-owned repositories. Set to null to disable.
30+
31+
# Rules to explicitly disallow force pushes and branch deletions
32+
# allow_force_pushes: false
33+
# allow_deletions: false

0 commit comments

Comments
 (0)