Skip to content
This repository was archived by the owner on Jun 15, 2022. It is now read-only.

Seekrets install - download and enable rulesets #83

Merged
merged 3 commits into from
Nov 14, 2016
Merged

Conversation

alain-hoang
Copy link
Contributor

This PR enables downloading and enabling of rulesets in the seekrets-install script.

@alain-hoang
Copy link
Contributor Author

The failures that are occurring in #69 are a result of a missing dependency. See #75 for more information regarding this. This PR also includes changes to address warnings issued from travis however the build failure requires a change in upstream. When the dependency has been fixed at upstream then the missing dependency build failures should stop occurring.

@alain-hoang alain-hoang mentioned this pull request Oct 27, 2016
11 tasks
@rogeruiz
Copy link
Contributor

@alain-hoang this PR is going to merge all of the changes in to master when it should be pointing to seekret instead.

@rogeruiz rogeruiz changed the base branch from master to seekret October 27, 2016 21:35
@alain-hoang
Copy link
Contributor Author

Thank you @rogeruiz for ensuring this pull request goes to the proper branch! I was under the impression that branching from the the intended branch seekret would work but not in this case.

git seekret --global hook --enable-all
# Activate all installed rules
git seekret rules | grep '\[ \] ' | awk ' { print $3 } ' | xargs -n 1 git seekret --global rules --enable
Copy link
Contributor

Choose a reason for hiding this comment

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

Given apuigsech/git-seekret#15 this can be simplified.

@LinuxBozo LinuxBozo merged commit 0b4d04d into seekret Nov 14, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants