Skip to content
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.

Sign up for GitHub

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

Jump to bottom

vigr: add page #8470

Merged
merged 2 commits into from
Sep 8, 2022
Merged

vigr: add page #8470

merged 2 commits into from
Sep 8, 2022

Conversation

F-Sossi
Copy link
Contributor

@F-Sossi F-Sossi commented Sep 7, 2022 •
edited
Loading

Added vigr command for modifying the /etc/group file

  • The page(s) are in the correct platform directories: common, linux, osx, windows, sunos, android, etc.
  • The page(s) have at most 8 examples.
  • The page description(s) have links to documentation or a homepage.
  • The page(s) follow the content guidelines.
  • The PR title conforms to the recommended templates.
  • Version of the command being documented (if known):

@F-Sossi
Create vigr.md
f223375 
Added vigr command for modifying the /etc/group file
@CLAassistant

This comment was marked as outdated.

Sign in to view
@github-actions github-actions bot added the new command Issues requesting creation of a new page. label Sep 7, 2022
@mfrw mfrw changed the title Create vigr.md vigr: add page Sep 7, 2022
navarroaxel
navarroaxel reviewed Sep 7, 2022
View reviewed changes
Copy link
Member

@navarroaxel navarroaxel left a comment

Choose a reason for hiding this comment

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

Hi @F-Sossi, welcome and thanks for your submission. I left a comment

pages/linux/vigr.md Outdated Show resolved Hide resolved
CleanMachine1
CleanMachine1 approved these changes Sep 7, 2022
View reviewed changes
Copy link
Member

@CleanMachine1 CleanMachine1 left a comment

Choose a reason for hiding this comment

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

Once axel's comment is applied

@F-Sossi @navarroaxel
Update pages/linux/vigr.md
9d473ef 
Co-authored-by: Axel Navarro <navarroaxel@gmail.com>
mfrw
mfrw approved these changes Sep 8, 2022
View reviewed changes
Copy link
Member

@mfrw mfrw left a comment

Choose a reason for hiding this comment

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

LGTM :)

Thank you @F-Sossi for your contribution 🥇

@mfrw mfrw merged commit 675613a into tldr-pages:main Sep 8, 2022
@vitorhcl vitorhcl mentioned this pull request Jan 30, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@navarroaxel navarroaxel navarroaxel left review comments

@mfrw mfrw mfrw approved these changes

@CleanMachine1 CleanMachine1 CleanMachine1 approved these changes

Assignees
No one assigned
Labels
new command Issues requesting creation of a new page.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
5 participants
@F-Sossi @CLAassistant @navarroaxel @mfrw @CleanMachine1