Skip to content

Latest commit

 

History

History
35 lines (28 loc) · 2.24 KB

COLLABORATING.md

File metadata and controls

35 lines (28 loc) · 2.24 KB

On how to work together

  • we do trunk-based development with a twist
    • Long lasting feature branches are discouraged, and instead small commits to main are preferred.
    • use preferably short-lived PRs when…
      • …you potentially want feedback on changes
      • …changes would break known downstream users, to allow those who are affected to influence the outcome towards greater value.
    • use issues for analysis and discussion of features and to track PRs…
      • …if collaboration on a feature and prior feedback is desired
    • feel free to use the project-board to organize your issues, PRs or cards
  • main must never be broken or show warnings
    • An easy way to achieve this is to run make tests check-size before pushing or make tests check-size && git push.
    • If you're unsure about remembering to do this, we suggest using a pre-commit git hook.
  • if main breaks on CI which can happen nonetheless
    • …and you do know the cause, please fix it immediately. If necessary by reverting the offending commit until a more durable fix is possible.
    • …and you do not know the cause, please open a PR to invite collaborators for their input. This is to avoid multiple collaborators trying to fix the issue independently, causing merge-conflicts and confusion. We use this PR as synchronization primitive.
  • for crates you own
    • feel free to make any kind of changes to it, including major ones.
    • please use cargo smart-release for publishing to crates.io as it will handle dependencies properly.
  • for crates you do not own
    • for major or architectural changes please open a [discussion], an issue or a PR to allow participation and don't merge until there is agreement.
    • for minor code changes, if they are minor, feel free to make any kind of change you need.

The workflow can be changed after public discussion - to get started, open a PR.

Please see the development guide for more detailed information on how code and cargo manifests are structured.