Skip to content

Commit ccebb4f

Browse files
committed
add notes
1 parent f2944cd commit ccebb4f

File tree

1 file changed

+23
-0
lines changed

1 file changed

+23
-0
lines changed

docs/maintainer-notes.md

Lines changed: 23 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,23 @@
1+
# Maintainer Notes
2+
3+
This is a living document for the maintainers of this repository.
4+
5+
## Deployment Process
6+
7+
This is a simple run down of how review / deploy a pull request.
8+
9+
1. A pull request is opened
10+
2. We review the code
11+
3. We comment `.deploy to development` and ensure the creator of the pull request is happy with the changes
12+
4. We comment `.deploy` to deploy to production. We "let it bake" and ensure everything is working as expected for a few minutes to a few days depending on the complexity of the changes.
13+
5. We approve the pull request and merge it into `main`
14+
15+
> It should be noted that the approval step can come before the deployment steps if that suits the situation better.
16+
17+
## CI Failures
18+
19+
A known issue (I am not sure of the cause) for CI failures is when dependabot opens a pull request. For some very strange reason, the necessary secrets are not injected into the Actions workflow when the pull request comes from dependabot. This causes the wrangler environment in CI to fail because it lacks the proper credentials to authenticate with Cloudflare.
20+
21+
The fix: Simply push a commit to the branch in question. It can be an empty commit if you like. This will trigger the CI workflow again and the secrets will be injected properly.
22+
23+
IDK why this happens, but it does. ¯\\\_(ツ)\_

0 commit comments

Comments
 (0)