Skip to content

Improve samples#52967

Open
ivanPeshterskii wants to merge 3 commits intodotnet:mainfrom
ivanPeshterskii:improve-samples
Open

Improve samples#52967
ivanPeshterskii wants to merge 3 commits intodotnet:mainfrom
ivanPeshterskii:improve-samples

Conversation

@ivanPeshterskii
Copy link
Copy Markdown

Summary

Add some comments, public properties and encapsulation in order to follow SOLID principles

@ivanPeshterskii ivanPeshterskii requested review from a team, BillWagner and adegeo as code owners April 10, 2026 07:48
@dotnetrepoman dotnetrepoman bot added this to the April 2026 milestone Apr 10, 2026
@dotnet-policy-service dotnet-policy-service bot added dotnet-fundamentals/svc dotnet-docker/subsvc community-contribution Indicates PR is created by someone from the .NET community. labels Apr 10, 2026
Copy link
Copy Markdown
Member

@BillWagner BillWagner left a comment

Choose a reason for hiding this comment

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

Thanks @ivanPeshterskii

I had a couple comments to consider before we merge this.

Comment on lines 12 to 14
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Please revert this change. It doesn't follow our coding guidelines. We prefer only using this. when necessary to disambiguate.

Comment on lines 17 to 37
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

If we're making updates, I'd change these to field backed auto-properties.

Refactor properties to use expression-bodied members and remove 'this' references for clarity.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

community-contribution Indicates PR is created by someone from the .NET community. dotnet-docker/subsvc dotnet-fundamentals/svc

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants