Skip to content

Add sample for tickerQ#4038

Open
AboubakrNasef wants to merge 4 commits intoBrighterCommand:masterfrom
AboubakrNasef:AddSampleForTickerQ
Open

Add sample for tickerQ#4038
AboubakrNasef wants to merge 4 commits intoBrighterCommand:masterfrom
AboubakrNasef:AddSampleForTickerQ

Conversation

@AboubakrNasef
Copy link
Contributor

No description provided.

Copy link

@codescene-delta-analysis codescene-delta-analysis bot left a comment

Choose a reason for hiding this comment

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

No application code in the PR — skipped Code Health checks.

See analysis details in CodeScene

Quality Gate Profile: Clean Code Collective
Install CodeScene MCP: safeguard and uplift AI-generated code. Catch issues early with our IDE extension and CLI tool.

Copy link
Member

@iancooper iancooper left a comment

Choose a reason for hiding this comment

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

Just one comment. Be grateful if you could fix.


namespace Greeting.Models
{
public class GreetingMapper : IAmAMessageMapper<GreetingEvent>
Copy link
Member

Choose a reason for hiding this comment

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

You can just use a default mapper now, and I am slowly trying to move our samples over, except for the ones that demonstrate explicit mapping, or transformer pipelines

@iancooper iancooper added 2 - In Progress .NET Pull requests that update .net code V10.X labels Mar 4, 2026
@iancooper
Copy link
Member

@AboubakrNasef Just checking if you saw the comment

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2 - In Progress .NET Pull requests that update .net code V10.X

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants