Skip to content

Issue / .NET 6 / Set Usage#60

Merged
cihandeniz merged 9 commits into
mainfrom
issue/net6/set-usage
Sep 21, 2022
Merged

Issue / .NET 6 / Set Usage#60
cihandeniz merged 9 commits into
mainfrom
issue/net6/set-usage

Conversation

@cihandeniz
Copy link
Copy Markdown
Contributor

@cihandeniz cihandeniz commented Sep 19, 2022 •
edited
Loading

Mümkün olan yerlerde List yerine Set kullanılarak performans iyileştirilecek

İşler

  • Model'lerde Marks'in List yerine Set olmasi
  • Marks'larda Any kaldi mi kontrol edilmesi
  • ConventionBasedCodingStyle'da types list degil set olsun, contains diyor
  • Diger List'lerin arastirilmasi
    • OPTIMIZED_TYPES'taki list degil set olsun

Ek İşler

  • implement Write_tests tests

@cihandeniz cihandeniz self-assigned this Sep 19, 2022
@cihandeniz cihandeniz marked this pull request as ready for review September 21, 2022 07:01
ozkanbicer78
ozkanbicer78 reviewed Sep 21, 2022
View reviewed changes
Comment thread samples/Routine.Samples.Client/Program.cs
ozkanbicer78
ozkanbicer78 approved these changes Sep 21, 2022
View reviewed changes
Copy link
Copy Markdown
Contributor

@ozkanbicer78 ozkanbicer78 left a comment

Choose a reason for hiding this comment

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

👍

@cihandeniz cihandeniz merged commit 719ddda into main Sep 21, 2022
@cihandeniz cihandeniz deleted the issue/net6/set-usage branch September 21, 2022 08:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ozkanbicer78 ozkanbicer78 ozkanbicer78 approved these changes

@oguzhanygt oguzhanygt Awaiting requested review from oguzhanygt

Assignees

@cihandeniz cihandeniz

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@cihandeniz @ozkanbicer78