Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
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
30 changes: 29 additions & 1 deletion .claude/rules/commit-conventions.md
Original file line number Diff line number Diff line change
@@ -1,4 +1,32 @@
# Commit Message Format
# Git Workflow & Commit Conventions

## Branching

**Always create new branches from `master`** (the main branch), not from feature branches:

```bash
# Correct workflow
git checkout master
git pull origin master
git checkout -b fix/my-fix

# Wrong - creates branch from current branch which may not be master
git checkout -b fix/my-fix
```

Before creating a PR branch, always:
1. Switch to `master`
2. Pull latest changes
3. Then create the new branch

## Never Push Directly to Master

**All changes must go through pull requests.** Never push commits directly to master, even for documentation changes. This ensures:
- Code review for all changes
- CI checks run before merge
- Clean git history with traceable PRs

## Commit Message Format

All commits should follow the Conventional Commits specification to enable automatic changelog generation.

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -92,7 +92,7 @@ public static async UniTask<SceneHandle> LoadHotUpdateScene(ResourcePackage pack
// Call error callback
await callbacks.OnError(e);
// Switch back to previous scene
await SceneManager.LoadSceneAsync(previousSceneName);
await SceneManager.LoadSceneAsync(previousSceneName).ToUniTask();

return null;
}
Expand Down
Loading