Conversation
The list lacks parallel structure. "Publishing new packages" uses a gerund, but "view and install existing packages" shifts to bare infinitives without "to," creating inconsistency.
There was a problem hiding this comment.
Pull Request Overview
This PR fixes a grammatical issue in the intro text for the GitHub Packages learning section by improving parallel structure in a list of actions.
- Fixed parallel structure by changing "view and install" to "viewing and installing" to match the gerund form used throughout the list
- Reorganized the phrase about deleting packages for better flow and clarity
How to review these changes 👓Thank you for your contribution. To review these changes, choose one of the following options: A Hubber will need to deploy your changes internally to review. Table of review linksNote: Please update the URL for your staging server or codespace. The table shows the files in the
Key: fpt: Free, Pro, Team; ghec: GitHub Enterprise Cloud; ghes: GitHub Enterprise Server 🤖 This comment is automatically generated. |
|
@iamdulanga Thanks for flagging this! I rearranged it a bit to flow better as long as we were here, in addition to your grammatical improvement. I'll get it added to the merge queue once checks are passing. |
|
Thanks very much for contributing! Your pull request has been merged 🎉 You should see your changes appear on the site in approximately 24 hours. If you're looking for your next contribution, check out our help wanted issues ⚡ |
The list lacks parallel structure. "Publishing new packages" uses a gerund, but "view and install existing packages" shifts to bare infinitives without "to," creating inconsistency.
Why:
Closes:
What's being changed (if available, include any code snippets, screenshots, or gifs):
Check off the following: