Skip to content
This repository has been archived by the owner on Jan 10, 2024. It is now read-only.

Make "Redirect Github webhooks" step mandatory #252

Merged
merged 1 commit into from
Oct 1, 2019
Merged

Make "Redirect Github webhooks" step mandatory #252

merged 1 commit into from
Oct 1, 2019

Conversation

MVrachev
Copy link
Contributor

In our doc "Setting up a manual deployment" we didn't emphasize
how important it is to redirect the GitHub webhooks to your local
machine.
Without this step you can't see how Precaution actually works!

Signed-off-by: Martin Vrachev mvrachev@vmware.com

@codecov-io
Copy link

codecov-io commented Sep 19, 2019

Codecov Report

Merging #252 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@          Coverage Diff           @@
##           master    #252   +/-   ##
======================================
  Coverage    98.3%   98.3%           
======================================
  Files          18      18           
  Lines         355     355           
  Branches       38      38           
======================================
  Hits          349     349           
  Misses          6       6

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 4dc6c38...a803a70. Read the comment docs.

docs/manual_deployment.md Outdated Show resolved Hide resolved
In our doc "Setting up a manual deployment" we didn't emphasize
how important it is to redirect the GitHub webhooks to your local
machine.
Without this step you can't see how Precaution actually works!

Signed-off-by: Martin Vrachev <mvrachev@vmware.com>
@joshuagl joshuagl merged commit a1f47ed into securesauce:master Oct 1, 2019
@MVrachev MVrachev deleted the change-manual-deployment-doc branch August 23, 2020 16:57
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