runtimejs / discussions

Random discussions about runtime.js
1 stars 1 forks source link

Affected subsystem in commit message? #5

Open facekapow opened 8 years ago

facekapow commented 8 years ago

I had an idea while commiting to the block subsystem on the disk-driver branch. How about adding the affected subsystem to the beginning of the commit message, the way node does? For example, [block] Minor code style fixes, or [kernel] Move js loader into kernel, add onexit/onerror/promise handlers, etc.

iefserge commented 8 years ago

Yeah good idea, more organized history is nice to have. Don't know how strict we should be with it though, since it's a bit of extra work to rename commits before merging a PR. I agree it would make it much easier to search commits.