Commits

Quick Rules

• Behavioral changes need a YouTrack ticket in the subject line.
• When a ticket ID is present, omit the subsystem prefix — the ticket provides sufficient context.
• Clearly non-behavioral changes may use a non-production label such as tests, cleanup, refactor, docs, format, style, setup, or misc.
• If there is any doubt whether the change is behavioral, do not use a non-production label.
• Write a full commit message (subject + body) for any non-trivial change. Subject-only is acceptable only for truly mechanical changes (typo, import, format).
• The body must explain why the change was made and summarize key decisions. Do not just list what files changed — the diff already shows that.
• Keep the first line concise; put rationale and important behavior notes in the body.
• If the user requests a suffix such as IJ-MR-100, put it in a final separate paragraph after a blank line.
• Do not use commits starting with WIP, fixup!, squash!, or amend!.

Examples

MRI-3589 harden single-flight recursion checks

Track active single-flight computations in coroutine context so recursive
awaits fail fast in both the owning coroutine and child coroutines.

IJ-MR-100

tests cidr: migrate JUnit 5 coverage

Convert remaining JUnit 4 test suites under cidr/coverage to JUnit 5.
Parametrized tests now use @MethodSource instead of Theories runner.

Anti-patterns

• Subject-only messages for non-trivial changes (even non-production ones).
• Restating the diff ("changed X in file Y") instead of explaining motivation.
• Using Conventional Commits format (fix(scope): ...).

References

• Commit Message Format (comprehensive)
• Online: YouTrack Article IJPL-A-217