Skip to content

cleanup: Typos, doc, and formatting cleanup in pip extension et al#1275

Merged
chrislovecnm merged 4 commits intobazel-contrib:mainfrom
rickeylev:multi.pip.tidying
Jun 20, 2023
Merged

cleanup: Typos, doc, and formatting cleanup in pip extension et al#1275
chrislovecnm merged 4 commits intobazel-contrib:mainfrom
rickeylev:multi.pip.tidying

Conversation

@rickeylev
Copy link
Copy Markdown
Collaborator

  • Corrects some typos in docs
  • Expands the user-facing documentation
  • Fixes errors having newlines in the middle of them
  • Renames some internal functions to be more self-describing.

* Corrects some typos in docs
* Expands the user-facing documentation
* Fixes errors having newlines in the middle of them
* Renames some internal functions to be more self-describing.
@rickeylev rickeylev requested a review from chrislovecnm June 15, 2023 22:46
@rickeylev rickeylev requested a review from hrfuller as a code owner June 15, 2023 22:46
Copy link
Copy Markdown
Contributor

@chrislovecnm chrislovecnm left a comment

Choose a reason for hiding this comment

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

More word smithing requested.

Copy link
Copy Markdown
Collaborator Author

@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.

ptal

Copy link
Copy Markdown
Contributor

@chrislovecnm chrislovecnm left a comment

Choose a reason for hiding this comment

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

LGTM

@chrislovecnm chrislovecnm enabled auto-merge June 20, 2023 19:57
@chrislovecnm chrislovecnm added this pull request to the merge queue Jun 20, 2023
Merged via the queue into bazel-contrib:main with commit cc43594 Jun 20, 2023
@rickeylev rickeylev deleted the multi.pip.tidying branch June 20, 2023 21:37
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