Skip to content

Conversation

chrislovecnm
Copy link
Contributor

This commit updates the README to meet the current bzlmod API. Some general tweaks and cleanup as well.

@chrislovecnm chrislovecnm requested a review from rickeylev as a code owner June 20, 2023 18:11
@chrislovecnm chrislovecnm force-pushed the readme-updates branch 4 times, most recently from 8820ef0 to 2574ee4 Compare June 20, 2023 18:33
Copy link
Collaborator

@rickeylev rickeylev left a comment

Choose a reason for hiding this comment

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

Minor things, really, otherwise lgtm.

@chrislovecnm
Copy link
Contributor Author

I decided to do a more aggressive edit and moved the README to here for editing and comments: https://docs.google.com/document/d/1rXCgjDVV9Uv3_-Uk4adqywWjT2pEn-GpHUGpCju6Uo8/edit

@chrislovecnm
Copy link
Contributor Author

@rickeylev PTAL at the google doc

@chrislovecnm chrislovecnm force-pushed the readme-updates branch 3 times, most recently from a290098 to 93c018f Compare July 23, 2023 16:00
Updating README to meet current bzlmod API.
General improvements with grammar and readability.
@chrislovecnm
Copy link
Contributor Author

@chrislovecnm PTAL

@chrislovecnm chrislovecnm added this pull request to the merge queue Jul 24, 2023
Merged via the queue into bazel-contrib:main with commit 23354a9 Jul 24, 2023
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