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

git-diff-files: add page #6896

Merged
merged 5 commits into from
Oct 21, 2021
Merged

Conversation

adipopbv
Copy link
Contributor

@adipopbv adipopbv commented Oct 8, 2021

  • The page (if new), does not already exist in the repository.
  • The page is in the correct platform directory (common/, linux/, etc.)
  • The page has 8 or fewer examples.
  • The PR title conforms to the recommended templates.
  • The page follows the content guidelines.
  • The page description includes a link to documentation or a homepage (if applicable).

Version of the command being documented (if known):

Tested with git 2.33.0

@CLAassistant

This comment has been minimized.

@pixelcmtd pixelcmtd added the new command Issues requesting creation of a new page or PRs adding a new page for a command. label Oct 8, 2021
Copy link
Collaborator

@marchersimon marchersimon left a comment

Choose a reason for hiding this comment

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

Even though you used the description from the docs, I don't think it's quite correct. It doesn't compare files like with git diff, but only shows their sha1 hashes and modes.

pages/common/git-diff-files.md Outdated Show resolved Hide resolved
Copy link
Collaborator

@marchersimon marchersimon left a comment

Choose a reason for hiding this comment

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

Thanks @adipopbv

@blueskyson blueskyson merged commit ddf49b9 into tldr-pages:main Oct 21, 2021
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 or PRs adding a new page for a command.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants