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

Made proper exception to gitleaks, fixing megalint #434

Merged
merged 1 commit into from
Aug 5, 2024

Conversation

BradleyOlson64
Copy link
Contributor

@BradleyOlson64 BradleyOlson64 commented Aug 2, 2024

Gitleaks thinks that a sudo public key in a chainspec is a leak. Had to add an exemption for the sudo key used in mainnet launch dry run 3

Also removed outdated exemptions corresponding to commits that were squashed.

@BradleyOlson64 BradleyOlson64 requested a review from a team as a code owner August 2, 2024 17:14
@BradleyOlson64 BradleyOlson64 requested review from atodorov and dave1374 and removed request for a team August 2, 2024 17:14
@BradleyOlson64 BradleyOlson64 changed the title Made proper exception to gitleaks fixing megalint Made proper exception to gitleaks, fixing megalint Aug 2, 2024
@BradleyOlson64 BradleyOlson64 requested review from beqaabu and rustlang-dev and removed request for dave1374 August 2, 2024 17:25
Copy link

github-actions bot commented Aug 2, 2024

For full LLVM coverage report click here!

@BradleyOlson64 BradleyOlson64 merged commit 0857733 into dev Aug 5, 2024
53 checks passed
@atodorov atodorov deleted the brad-fix-gitleaks branch August 7, 2024 07:30
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.

3 participants