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.

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

yadm: add pages #13920

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

yadm: add pages #13920

wants to merge 1 commit into from

Conversation

louiss0
Copy link

@louiss0 louiss0 commented Oct 1, 2024

  • 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 page(s) follow the style guide.
  • The PR title conforms to the recommended templates.
  • Version of the command being documented (if known):

@github-actions github-actions bot added the new command Issues requesting creation of a new page. label Oct 1, 2024
@louiss0
Copy link
Author

louiss0 commented Oct 1, 2024

Well I hope this one works now. Creating a new PR was the only way to get rid of the wrong author.

@sebastiaanspeck sebastiaanspeck changed the title yadm: add new pages yadm: add pages Oct 1, 2024
Copy link
Member

@sebastiaanspeck sebastiaanspeck left a comment

Choose a reason for hiding this comment

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

Thank you for your first contribution. Please make sure to read the style guide and contributing guide.

I have added some comments that need to be addressed before the PR is GTG.

@@ -0,0 +1,9 @@
# yadm-bootstrap

> This command allows you to execute Yadm's bootstrap file.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
> This command allows you to execute Yadm's bootstrap file.
> Execute Yadm's bootstrap file.


- Create the necessary files and folders for encryption:

`touch {{encrypt-file-path}} && mkdir {{archive-folder-path}}`
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
`touch {{encrypt-file-path}} && mkdir {{archive-folder-path}}`
`touch {{path/to/encrypt_file}} && mkdir {{path/to/archive_folder}}`


- Share the repository using GPG:

`yadm git-crypt add-gpg-user {{USER_ID}}`
Copy link
Member

Choose a reason for hiding this comment

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

Does the USER_ID need to be capitalised?

# yadm-perms

> Update permissions.
> It is usually unnecessary to run this command, as `yadm` automatically processes permissions by default. This automatic behavior can be disabled by setting the configuration `yadm.auto-perms` to "false".
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
> It is usually unnecessary to run this command, as `yadm` automatically processes permissions by default. This automatic behavior can be disabled by setting the configuration `yadm.auto-perms` to "false".
> It is usually unnecessary to run this command, as `yadm` automatically processes permissions by default. This automatic behavior can be disabled by setting the configuration `yadm.auto-perms` to `"false"`.

- Set the symmetric cipher to utilize for encryption:

`yadm transcrypt --cipher={{cipher}}`
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
`yadm transcrypt --cipher={{cipher}}`
`yadm transcrypt --cipher={{cipher}}`


- Override the location of the yadm encrypted files archive:

`yadm --yadm-archive`
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
`yadm --yadm-archive`
`yadm --yadm-archive`


> A dotfiles manager that works by using `git`.
> The subcommands are just like `git` but use `yadm`.
> You can use `yadm` (init | clone | push | pull).
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
> You can use `yadm` (init | clone | push | pull).
> Some subcommands such as `init`, `clone`, `push`, and `pull` have their own usage documentation.

https://github.com/tldr-pages/tldr/blob/main/CONTRIBUTING.md#referencing-subcommands

@@ -0,0 +1,13 @@
# yadm-upgrade

> Use this command upgrade `yadm` to the latest version.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
> Use this command upgrade `yadm` to the latest version.
> Upgrade `yadm` to the latest version.

> Upgrading will attempt to de-initialize and re-initialize your submodules.
> More information: <https://github.com/TheLocehiliosan/yadm/blob/master/yadm.md#commands>.
- Upgrade:
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- Upgrade:
- Upgrade `yadm` to the latest version:

@@ -0,0 +1,13 @@
# yadm-enter

> Run a sub-shell with all Git variables set. This sub-shell can be used to easily interact with your `yadm` repository using Git commands.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
> Run a sub-shell with all Git variables set. This sub-shell can be used to easily interact with your `yadm` repository using Git commands.
> Run a sub-shell with all Git variables set. This sub-shell can be used to easily interact with the local `yadm` repository using Git commands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
new command Issues requesting creation of a new page.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants