Skip to content

Commit

Permalink
add notes
Browse files Browse the repository at this point in the history
  • Loading branch information
GrantBirki committed Jun 18, 2023
1 parent f2944cd commit ccebb4f
Showing 1 changed file with 23 additions and 0 deletions.
23 changes: 23 additions & 0 deletions docs/maintainer-notes.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,23 @@
# Maintainer Notes

This is a living document for the maintainers of this repository.

## Deployment Process

This is a simple run down of how review / deploy a pull request.

1. A pull request is opened
2. We review the code
3. We comment `.deploy to development` and ensure the creator of the pull request is happy with the changes
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.
5. We approve the pull request and merge it into `main`

> It should be noted that the approval step can come before the deployment steps if that suits the situation better.
## CI Failures

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.

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.

IDK why this happens, but it does. ¯\\\_(ツ)\_

0 comments on commit ccebb4f

Please sign in to comment.