-
-
Notifications
You must be signed in to change notification settings - Fork 6.6k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Event Model Diagram #5861
base: develop
Are you sure you want to change the base?
Event Model Diagram #5861
Conversation
|
❌ Deploy Preview for mermaid-js failed.
|
commit: |
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #5861 +/- ##
==========================================
- Coverage 5.00% 4.99% -0.01%
==========================================
Files 338 340 +2
Lines 48221 48282 +61
Branches 551 553 +2
==========================================
Hits 2413 2413
- Misses 45808 45869 +61
Flags with carried forward coverage won't be shown. Click here to find out more.
|
The latest updates on your projects. Learn more about Argos notifications ↗︎ Waiting for the first build to start… |
📑 Summary
Implements an event model diagram. Creating this PR early to make sure I'm not going off in some ridiculous direction, given that I haven't contributed here before.
Resolves #5860
📏 Design Decisions
Trying to follow the steps here as closely as possible. The grammar is taken from my high level description in the issue thread.
📋 Tasks
Make sure you
MERMAID_RELEASE_VERSION
is used for all new features.pnpm changeset
and following the prompts. Changesets that add features should beminor
and those that fix bugs should bepatch
. Please prefix changeset messages withfeat:
,fix:
, orchore:
.