aboutsummaryrefslogtreecommitdiffstats
path: root/DEVNOTES.md
diff options
context:
space:
mode:
authorAlex Schofield <git@ajschof.me>2024-08-12 10:43:23 +0100
committerAlex Schofield <git@ajschof.me>2024-08-12 10:43:23 +0100
commit19cd19b68cbd063ced675d63eab6b49983be89d9 (patch)
tree0609b78d9faa03a453864c93e327fe7cd045c7e7 /DEVNOTES.md
parentae67733e93d9eac40a98c03c011dfc506b40d521 (diff)
downloadde-project-bentley-19cd19b68cbd063ced675d63eab6b49983be89d9.tar.gz
de-project-bentley-19cd19b68cbd063ced675d63eab6b49983be89d9.zip
docs: add markdown extension
Diffstat (limited to 'DEVNOTES.md')
-rw-r--r--DEVNOTES.md55
1 files changed, 55 insertions, 0 deletions
diff --git a/DEVNOTES.md b/DEVNOTES.md
new file mode 100644
index 0000000..00f06ec
--- /dev/null
+++ b/DEVNOTES.md
@@ -0,0 +1,55 @@
+# Workflow
+
+## Commits
+
+### Make small and focused commits
+- Please avoid mixing unrelated changes in a single commit
+- Commit at regular points to revert changes easily if needed
+
+### Write clear commit messages
+- Limit subject line to 50 characters
+- Provide more detailed explainations in the commit body (if required)
+- Use the imperative mood in the subject line (e.g. 'add' instead of 'added')
+
+```
+$ ~ git commit
+```
+
+```
+[Type]: [Short Subject]
+---[Blank Line]---
+[Body, Limit to 72 Characters]
+```
+- Types: feat, fix, docs, style, refactor, test, chore, ci, perf
+ - See [here](https://eagerworks.com/blog/conventional-commits) for more information
+
+## Branches
+
+### Naming Conventions
+
+- Use lowercase with hyphens
+- Include type and change with small description
+
+```
+[type]/[brief-description] :: e.g. feature/api
+```
+### Base Branch
+
+- Branch from `develop` for features and non-urgent fixes
+- Branch off from `main` for urgent changes (project deadline) - this should be rarely used
+
+### Keep branches updated
+
+- Regularly merge and also delete branches when stale
+
+## PRs
+
+1. Create a pull request for each feature or fix (link to related issues)
+2. Write a clear description which...
+ 1. Summarises the changes
+ 2. Explains the reasoning behind the changes
+ 3. Lists any areas of concerns (i.e. breaking changes)
+3. Keep PRs focused - split changes into multiple PRs if needed
+4. Assign someone to review
+5. Merge ONLY after team approval - resolve conflicts & ensure CI checks pass
+6. Use [squash and merge](https://learn.microsoft.com/en-us/azure/devops/repos/git/merging-with-squash?view=azure-devops) when needed to keep main branch history clean \ No newline at end of file
git.ajschof.me — hosted by ajschofield — powered by cgit