Skip to content

doc: streamline Matrix argument variants#6361

Merged
fingolfin merged 1 commit intomasterfrom
mh/Matrix-doc
May 4, 2026
Merged

doc: streamline Matrix argument variants#6361
fingolfin merged 1 commit intomasterfrom
mh/Matrix-doc

Conversation

@fingolfin
Copy link
Copy Markdown
Member

This should make it a bit clearer when a matrix object is used for data, and when as an "example" or "reference" object. And by merging some entries, I also think it is a bit easier to understand the relations.

However, to really make this good, examples for all or at leas most of the variants ought to be added...

@fingolfin fingolfin added the topic: documentation Issues and PRs related to documentation label May 3, 2026
@fingolfin fingolfin added the release notes: not needed PRs introducing changes that are wholly irrelevant to the release notes label May 3, 2026
@fingolfin fingolfin marked this pull request as ready for review May 3, 2026 23:12
@codecov
Copy link
Copy Markdown

codecov Bot commented May 4, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.66%. Comparing base (fa26016) to head (57474df).
⚠️ Report is 4 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #6361      +/-   ##
==========================================
+ Coverage   78.63%   78.66%   +0.02%     
==========================================
  Files         684      684              
  Lines      292693   292720      +27     
  Branches     8660     8660              
==========================================
+ Hits       230159   230264     +105     
+ Misses      60726    60642      -84     
- Partials     1808     1814       +6     

☔ 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.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Copy link
Copy Markdown
Contributor

@ThomasBreuer ThomasBreuer left a comment

Choose a reason for hiding this comment

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

The new description is better than the old one.
A few cross-references point to Matrix with label "for a list and a matrix object", this label does no longer exist.

@fingolfin fingolfin enabled auto-merge (squash) May 4, 2026 19:40
@fingolfin fingolfin merged commit 831d72d into master May 4, 2026
33 checks passed
@fingolfin fingolfin deleted the mh/Matrix-doc branch May 4, 2026 21:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release notes: not needed PRs introducing changes that are wholly irrelevant to the release notes topic: documentation Issues and PRs related to documentation

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants