-
Notifications
You must be signed in to change notification settings - Fork 39
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Showing
1 changed file
with
51 additions
and
54 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
8ce7ee5
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.
@12rambau and @dfguerrerom:
This should be a PR draft on a separate branch. Can you please update? I will be editing the pages in phases, which will require reviewing PR drafts again before submitting the actual PRs.
Thank you!
Alex
8ce7ee5
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.
hey @alexryangregor, the main branch wasn't protected to avoid direct pushes, I have protected it and I'm running the tests on it (with your latest changes), I hope there's nothing wrong.
8ce7ee5
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.
Thanks, @dfguerrerom. Only minor changes were made, mostly correcting grammatical errors, updating presentation for consistency across pages, and aligning with the style guide. I will be careful to avoid this in the future. My apologies again for any inconvenience. As @12rambau knows, I'm still new to GitHub, so if you notice anything I've done wrong, don't hesitate to let me know. I will continue editing the pages when I have breaks from more urgent tasks with the team. Thanks again!
8ce7ee5
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.
You committed this 8ce7ee5 directly to
main branch
, there was only one file modifieddocs/source/cli/gdrive.rst
so not big deal. As mentioned before, I have protected the main branch to avoid this, and in the future, you won't be able to do that (directly modify the main branch) without a PR.