Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Tag protocol-definitions and server types necessary for IQuorum to be alpha #18634

Merged

Conversation

anthony-murphy
Copy link
Contributor

This PR is preparation for changing the tagging on the IQuorum to be alpha. All these types were automatically identified as transitive dependency of the IQuorum class which will be made alpha in a later review.

We will stage these commits and invite review, however review items will generally be added to the backlog as future items, as these type updates are necessary for the current state of the code base.

@anthony-murphy anthony-murphy requested review from a team as code owners December 4, 2023 23:09
@github-actions github-actions bot added public api change Changes to a public API community-contribution base: main PRs targeted against main branch labels Dec 4, 2023
@anthony-murphy anthony-murphy added the fluid2alpha Tag to track discussions related to the Fluid 2.0 release label Dec 4, 2023
@@ -150,7 +150,7 @@ export interface IProtocolState {
}

/**
* @internal
* @alpha
Copy link
Contributor

Choose a reason for hiding this comment

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

Any docs we can add while we're here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

i'm not sure what this is or how to describe it

@anthony-murphy anthony-murphy merged commit 5082dd0 into microsoft:main Dec 4, 2023
24 checks passed
@anthony-murphy anthony-murphy deleted the beta-loader-protocol-defs branch December 4, 2023 23:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
base: main PRs targeted against main branch community-contribution fluid2alpha Tag to track discussions related to the Fluid 2.0 release public api change Changes to a public API
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants