-
-
Notifications
You must be signed in to change notification settings - Fork 28.1k
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
Add Pentest Cheat Sheets #3068
Add Pentest Cheat Sheets #3068
Conversation
Added link to awesome-pentest-cheat-sheets
Thanks for making an Awesome list! 🙌 It looks like you didn't read the guidelines closely enough. I noticed multiple things that are not followed. Try going through the list point for point to ensure you follow it. I spent a lot of time creating the guidelines so I wouldn't have to comment on common mistakes, and rather spend my time improving Awesome. |
It appears that many of your items do not have any description. You also need to review at least 2 pull requests before making your own pull request. Additionally running the linter on your repo, the following errors are given:
For the first error just make your title for the header The double-link errors can be easily solved by changing the links to Then the last issue needs to be fixed too. (no-inline-padding) |
readme.md
Outdated
@@ -650,6 +650,7 @@ | |||
## Security | |||
|
|||
- [Application Security](https://github.com/paragonie/awesome-appsec#readme) | |||
- [Awesome Pentest Cheat Sheets](https://github.com/ByteSnipers/awesome-pentest-cheat-sheets) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The URL to your list should end in #readme
.
- [Awesome Pentest Cheat Sheets](https://github.com/ByteSnipers/awesome-pentest-cheat-sheets) | |
- [Awesome Pentest Cheat Sheets](https://github.com/ByteSnipers/awesome-pentest-cheat-sheets#readme) |
I've update the repository, but I still have some issues, what I cannot solve, because I cannot identify why they are issues. ✖ 15:1 ToC item "Privilege Escalation" does not match corresponding heading "Web Pentesting" remark-lint:awesome-toc |
I run your repo in Linting and can see some errors are still need to be resolved. also you have no pull requests reviews links so I suggest you reading through it one more time! :) cc. @sindresorhus |
Apart from fixing liniting issues I suggest you add a books sections and books related to book bountry web penetration testing mobile app pentesting....etc |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Add it to the end of the list
As per the guidelines, the title of the pull request should be "Add Pentest Cheat Sheets". |
f0658c5
to
993cee4
Compare
Added Awesome Pentest Cheat Sheets
URL: https://github.com/ByteSnipers/awesome-pentest-cheat-sheets
Awesome Pentest Cheat Sheets contains a lot of security and pentesting related information. The list has a huge list of very sorted and selected resources, which can save a lot of time to any security oriented people. This list was started 2017 and was improved over the years. The initial awesome pentest cheat sheets collects over 3.8k stars