Skip to content

Update diagrams and links in README#4397

Merged
danbarr merged 2 commits intomainfrom
update-diagrams
Mar 27, 2026
Merged

Update diagrams and links in README#4397
danbarr merged 2 commits intomainfrom
update-diagrams

Conversation

@danbarr
Copy link
Copy Markdown
Collaborator

@danbarr danbarr commented Mar 26, 2026

Overview

Refreshed diagrams used in the README using new designs.

Also updated some links to not rely on redirects.

Signed-off-by: Dan Barr 6922515+danbarr@users.noreply.github.com

Large PR Justification

  • Changed lines count is artificially inflated by SVG file replacements.

danbarr added 2 commits March 26, 2026 19:49
Signed-off-by: Dan Barr <6922515+danbarr@users.noreply.github.com>
Signed-off-by: Dan Barr <6922515+danbarr@users.noreply.github.com>
@danbarr danbarr requested a review from JAORMX as a code owner March 26, 2026 23:51
@github-actions github-actions bot added the size/XL Extra large PR: 1000+ lines changed label Mar 26, 2026
Copy link
Copy Markdown
Contributor

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Large PR Detected

This PR exceeds 1000 lines of changes and requires justification before it can be reviewed.

How to unblock this PR:

Add a section to your PR description with the following format:

## Large PR Justification

[Explain why this PR must be large, such as:]
- Generated code that cannot be split
- Large refactoring that must be atomic
- Multiple related changes that would break if separated
- Migration or data transformation

Alternative:

Consider splitting this PR into smaller, focused changes (< 1000 lines each) for easier review and reduced risk.

See our Contributing Guidelines for more details.


This review will be automatically dismissed once you add the justification section.

@codecov
Copy link
Copy Markdown

codecov bot commented Mar 26, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 69.51%. Comparing base (8fc2c71) to head (a6c0b94).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4397      +/-   ##
==========================================
+ Coverage   69.47%   69.51%   +0.04%     
==========================================
  Files         485      485              
  Lines       49805    49805              
==========================================
+ Hits        34603    34624      +21     
+ Misses      12523    12503      -20     
+ Partials     2679     2678       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@github-actions github-actions bot added size/XL Extra large PR: 1000+ lines changed and removed size/XL Extra large PR: 1000+ lines changed labels Mar 27, 2026
@github-actions github-actions bot dismissed their stale review March 27, 2026 00:25

Large PR justification has been provided. Thank you!

@github-actions
Copy link
Copy Markdown
Contributor

✅ Large PR justification has been provided. The size review has been dismissed and this PR can now proceed with normal review.

@danbarr danbarr merged commit bc24877 into main Mar 27, 2026
38 checks passed
@danbarr danbarr deleted the update-diagrams branch March 27, 2026 00:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/XL Extra large PR: 1000+ lines changed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants