Skip to content

Refactor ProviderVisitor to generic form#275

Merged
alex-kulakov merged 4 commits intoDataObjects-NET:masterfrom
servicetitan:upstream/RefactorProviderVisitor
Jul 28, 2023
Merged

Refactor ProviderVisitor to generic form#275
alex-kulakov merged 4 commits intoDataObjects-NET:masterfrom
servicetitan:upstream/RefactorProviderVisitor

Conversation

@SergeiPavlov
Copy link
Copy Markdown
Contributor

@SergeiPavlov SergeiPavlov commented Aug 27, 2022

  • Remove duplicated switch statement and convert it to "switch expression"
  • Remove unnecessary casts to CompilableProvider

alex-kulakov
alex-kulakov previously approved these changes Jul 27, 2023
Copy link
Copy Markdown
Contributor

@alex-kulakov alex-kulakov left a comment

Choose a reason for hiding this comment

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

Everything is OK but conflicts should be resolved

@SergeiPavlov
Copy link
Copy Markdown
Contributor Author

Everything is OK but conflicts should be resolved

done

@alex-kulakov alex-kulakov merged commit cca10f7 into DataObjects-NET:master Jul 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants