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

Update README.rst #209

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

Conversation

AnirudhMergu
Copy link

Issue #, if available:

  • Updated the wordings in README.rst

Description of changes:
Before

Before making public a repository

After

Before making a repository public

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@@ -106,7 +106,7 @@ Add custom providers to scan for security credentials.
git secrets --add-provider -- cat /path/to/secret/file/patterns


Before making public a repository
Before making a repository public
---------------------------------

With git-secrets is also possible to scan a repository including all revisions:
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
With git-secrets is also possible to scan a repository including all revisions:
With git-secrets it is also possible to scan a repository including all revisions:

While we're in the neighborhood.

@sparr
Copy link
Contributor

sparr commented Jun 21, 2023

Changes to README.rst also require make man to rebuild the man page.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants