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

fetchgit: remove "security" from comment about "security risk" #185005

Merged
merged 1 commit into from Jan 14, 2023
Merged

fetchgit: remove "security" from comment about "security risk" #185005

merged 1 commit into from Jan 14, 2023

Conversation

ghost
Copy link

@ghost ghost commented Aug 3, 2022

Description of changes

Closes #178410

Things done

@ofborg ofborg bot added 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild 10.rebuild-linux: 0 This PR does not cause any packages to rebuild labels Aug 3, 2022
@bobby285271 bobby285271 added the 12.approvals: 1 This PR was reviewed and approved by one reputable person label Aug 4, 2022
@mweinelt mweinelt added 12.approvals: 2 This PR was reviewed and approved by two reputable people and removed 12.approvals: 1 This PR was reviewed and approved by one reputable person labels Aug 5, 2022
@Artturin Artturin merged commit 1121c8f into NixOS:master Jan 14, 2023
@ghost ghost deleted the pr/fetchgit/fix-comment branch January 15, 2023 16:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
6.topic: fetch 10.rebuild-darwin: 0 This PR does not cause any packages to rebuild 10.rebuild-linux: 0 This PR does not cause any packages to rebuild 12.approvals: 2 This PR was reviewed and approved by two reputable people
Projects
None yet
Development

Successfully merging this pull request may close these issues.

"fetching arbitrary (maybe dangling) commits may be a security risk"
5 participants