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

Deprecate connectrpc es and query-es #1597

Merged
merged 3 commits into from
Nov 20, 2024
Merged

Deprecate connectrpc es and query-es #1597

merged 3 commits into from
Nov 20, 2024

Conversation

mfridman
Copy link
Member

@mfridman mfridman commented Nov 20, 2024

This PR marks the following plugins as deprecated with a new description:

And updates the description for:

After this is merged/released, I'll make them as disabled: true in the source. I don't recall if we use that for any logic outside the fetcher.

Copy link
Member

@stefanvanburen stefanvanburen left a comment

Choose a reason for hiding this comment

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

seems good, just not sure about the markdown in the description as I didn't see other descriptions w/ markdown in them?

@@ -3,7 +3,7 @@ name: buf.build/connectrpc/es
plugin_version: v1.6.1
source_url: https://github.com/connectrpc/connect-es
integration_guide_url: https://connectrpc.com/docs/web/getting-started
description: Generates code that is compatible with browsers and Node.js for working with Connect, gRPC, and gRPC-Web.
description: Generates code that is compatible with browsers and Node.js for working with Connect, gRPC, and gRPC-Web. This plugin is deprecated, see [Migration Guide](https://github.com/connectrpc/connect-es/blob/v2.0.0/MIGRATING.md) to upgrade to the latest version.
Copy link
Member

Choose a reason for hiding this comment

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

Does the BSR's description field render markdown appropriately?

Copy link
Member Author

@mfridman mfridman Nov 20, 2024

Choose a reason for hiding this comment

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

It does not, I'll back out the links for now. But it'd be super handy to be able to link out, although you could argue that's the purpose of the "documentation" or "readme" section we'll be adding.

@mfridman mfridman requested a review from timostamm November 20, 2024 15:44
@mfridman mfridman merged commit 3424e6d into main Nov 20, 2024
6 checks passed
@mfridman mfridman deleted the mf/deprecate-connect-es branch November 20, 2024 15:56
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