You are viewing a plain text version of this content. The canonical link for it is here.

Posted to gitbox@yetus.apache.org by GitBox <gi...@apache.org> on 2022/08/29 15:39:03 UTC

[GitHub] [yetus] aw-was-here commented on a diff in pull request #286: YETUS-1194. upgrade Apache Creadur Rat project

aw-was-here commented on code in PR #286:
URL: https://github.com/apache/yetus/pull/286#discussion_r957498095


##########
.yetus/detsecrets-ignored-hashes.txt:
##########
@@ -26,3 +26,4 @@ d8e18255635efa84e79bb9eaf2ffad3371a95288
 3b3c84f10a63cb8e95b93ea5f46e02d885153f5d
 c267b646441a206d44803d8cb20896c4a166cac2
 f4d95a7d14e9c4c69afe28d54743222acbe04b74
+66defd532d6875b42111a786d7bf52bb8db61602

Review Comment:
   > What is this file?
   
   It is the exemption file for [detect-secrets](https://yetus.apache.org/documentation/0.14.0/precommit/plugins/detsecrets/). For Yetus, this mostly contains shas of other shas that detect-secrets finds.  I think there are some references to password variables in precommit as well.
   
   > Do we ever remove entries from it?
   
   We haven't yet, but we could.  It was just introduced as a new feature in 0.14 so shiny new.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: gitbox-unsubscribe@yetus.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org