-
Notifications
You must be signed in to change notification settings - Fork 135
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
Broken link in Editions Overview to "required considered harmful". #206
Comments
Thanks for raising this issue. That link shouldn't be there (it points to a topic that has internal-only guidance). I've sent an update out for review that removes the link. It should be pushed to the external site within the next couple of weeks. |
As well as appearing in protocolbuffers.github.io/content/overview.md Line 236 in b9be4af
|
Ah, yes. I'll remove that one, too. Thanks! |
Updates are made. Thanks again for raising the issue. |
https://github.com/protocolbuffers/protocolbuffers.github.io/blame/41c8ea607ec1e444a6480a9283882d12e0fa6da3/content/editions/overview.md#L337
This should probably link to https://protobuf.dev/programming-guides/dos-donts/#add-required.
The text was updated successfully, but these errors were encountered: