-
Couldn't load subscription status.
- Fork 1.4k
Update officials.md #5216
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
Update officials.md #5216
Conversation
Update docs to reflect the renaming of zio-web to zio-microservice (also mention other protocols in the summary)
| - [ZIO Akka Cluster](https://github.com/zio/zio-akka-cluster) — A ZIO wrapper for Akka Cluster | ||
| - [ZIO Config](https://github.com/zio/zio-config) — A ZIO based configuration parsing library | ||
| - [ZIO Web](https://github.com/zio/zio-web) — A ZIO-powered HTTP server and client | ||
| - [ZIO Microservice](https://github.com/zio/zio-microservice) — A ZIO-powered microservices via HTTP and other protocols |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
| - [ZIO Microservice](https://github.com/zio/zio-microservice) — A ZIO-powered microservices via HTTP and other protocols | |
| - [ZIO Microservice](https://github.com/zio/zio-microservice) — ZIO-powered microservices via HTTP and other protocols |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you also alphabetize?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@adamgfraser I kept the A for consistency with the other entries in the list. They all start with A. So I would suggest all are dropped or we keep this one as well.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Updated the order
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mkows As currently written it is grammatically incorrect because "microservices" is being used as an adjective and there is no noun.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Right. Good point. Updated.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you! 🙏
Just a minor comment on grammar.
|
Thanks @adamgfraser for reviewing! |
Update docs to reflect the renaming of zio-web to zio-microservice (also mention other protocols in the summary)