Skip to content

Add profiling merge example without arguments#846

Merged
fsimonis merged 2 commits intomasterfrom
add-argumentless-merge-ex
May 5, 2026
Merged

Add profiling merge example without arguments#846
fsimonis merged 2 commits intomasterfrom
add-argumentless-merge-ex

Conversation

@fsimonis
Copy link
Copy Markdown
Member

The first example used merge with arguments, which can be misunderstood to be the Participants to merge.

We now

  • first show the non-argument version that should be enough in most cases.
  • then explain what arguments mean
  • and finally show the example with arguments

@fsimonis fsimonis requested a review from carme-hp March 11, 2026 14:55
Copy link
Copy Markdown
Member

@carme-hp carme-hp left a comment

Choose a reason for hiding this comment

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

Thanks! I'd just asked you to add more on the default.

Comment thread content/docs/tooling/tooling-performance-analysis.md Outdated
@MakisH MakisH added the content Content-only issues label Mar 13, 2026
@fsimonis fsimonis force-pushed the add-argumentless-merge-ex branch from a3c318b to fcd61d8 Compare May 5, 2026 08:34
@fsimonis fsimonis enabled auto-merge (squash) May 5, 2026 08:34
@fsimonis fsimonis merged commit 4c6a3fd into master May 5, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

content Content-only issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants