Staging and Production Overview #7
vlazed
announced in
Developers
Replies: 0 comments
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
Diagram
graph TD; subgraph SubGraphA[Forks] A[Fork 1] --- B; B[Fork 2] -.- C[Fork N]; end SubGraphA <-->|pushes to/pulls from| D[Peak Inc SMH]; D -->|pushes to| E[Winded SMH]; E[Winded SMH] -->|push| F([Workshop Main Version]); D -->|pushes to| G([Workshop Beta Version]);Notation
Pushing / Pulling Method
Each fork pulls from Peak Inc SMH. To make changes to Peak Inc SMH, the fork should
This results in a clean history where the details of such changes can be viewed under the PR commit message. In addition, the PRs are reflected over in the workshop's changelog. Squashing and detailing the changes in a single PR commit reduces the volume of commits in the commit history.
Let's make the changes incremental. If you plan to make a big change to a subsystem of SMH, you should PR changes by part (see #24 and #29). It's harder to test a big PR, especially if it rewrites an existing workflow. If several subsystems within the PR breaks an existing workflow or introduces bugs, then the developer would have to look for particular commit/subsystem that broke, while also ensuring other subsystems are not broken; this increases development time and introduces delays as a consequence. By limiting the size of a PR, this ensures that each part of the PR gets completely tested in the workshop beta version.
Finalizing Squashes in PR
When pushing a PR (or any commit) to this repo, a workflow activates to automatically update the beta version. It transforms the commit message form Markdown to Steam BBCode. As of 7/6/2026, some Markdown formatting can reduce readability and also produce incomplete change notes. To reduce this, observe the following rules for the PR commit message:
Inline coderesolves to code blocks. Use italics insteadBeta Was this translation helpful? Give feedback.
All reactions