Skip to content

allow generate API for Multi content types of internal operation#2706

Merged
msyyc merged 4 commits intomainfrom
multi-content-types-internal
Jul 24, 2024
Merged

allow generate API for Multi content types of internal operation#2706
msyyc merged 4 commits intomainfrom
multi-content-types-internal

Conversation

@msyyc
Copy link
Copy Markdown
Member

@msyyc msyyc commented Jul 18, 2024

fix #1526

Today, when we see multiple content-type, we don't generate anything. I believe it's hard on people, and we should do the following:

  • If access is public (default), generate nothing
  • If access is private, generate the code as best as we can
    This way when people patch the code, they can at least call the private method

@msyyc msyyc marked this pull request as ready for review July 19, 2024 02:18
@msyyc msyyc enabled auto-merge (squash) July 24, 2024 02:13
@msyyc msyyc merged commit db8bda9 into main Jul 24, 2024
@msyyc msyyc deleted the multi-content-types-internal branch July 24, 2024 02:51
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.

[TypeSpec] Multiple content types should generate a private method

2 participants