-
-
Notifications
You must be signed in to change notification settings - Fork 27.8k
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 Hackathon #2433
Add Hackathon #2433
Conversation
unicorn |
Here are a few comments:
Apart from that, nice job! The list looks very useful. PS: What is that "unicorn" thing that everybody is commenting? |
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. |
Great feedback! I've revised the list to confirm even more closely to the guidelines (I read them carefully, trust me, but there are a lot of things to think about!) In addition to revisions to the list, I have moved the location of the entry to the arguably more appropriate 'Security' section. Please add yourself as reviewer to approve the Pull Request (top right of the page) - thanks! |
Not completely there yet: "Your entry should be added at the bottom of the appropriate category." |
Moved back to previous spot
🙈 Reverted back to previous location. Thx @steven2358 |
Woo, thank you. By the way, the 'unicorn' is one of the things the
guidelines ask you to do..
…On Tue, Dec 6, 2022 at 1:39 AM Steven Van Vaerenbergh < ***@***.***> wrote:
***@***.**** approved this pull request.
—
Reply to this email directly, view it on GitHub
<#2433 (review)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAAHYS7FDGJ7BFMEOK3L4FTWL2DLZANCNFSM6AAAAAASGXT4XM>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
This is my first review of an Awesome list, so I may not be the best at judging. |
this is a fantastic list! definitely a topic that i'm very interested in, so i'm sure i'll be using it's resources in the near future. the only "style" comment I would make is to: use
otherwise, really terrific! 👍 |
Thanks @Symbitic @modenero and @andrzejsydor I've incorporated your content and formatting suggestions. |
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.
👍
https://github.com/dribdat/awesome-hackathon
Tips and tools for running tolerant and productive hackathons. We started this list a year ago to connect with other hackathon organizers. In my opinion, hackathons are placed perfectly at the intersection of awesome "Learning" and "Security" topics.
By submitting this pull request I confirm I've read and complied with the below requirements 🖖
Add Name of List
. ✔️#readme
. ✔️Requirements for your Awesome list
awesome-lint
on your list and fix the reported issues. ✔️main
, notmaster
. ✔️awesome-name-of-list
. ✔️# Awesome Name of List
. ✔️awesome-list
&awesome
as GitHub topics. I encourage you to add more relevant topics. ✔️Footnotes
section at the bottom of the readme. The section should not be present in the Table of Contents. ✔️You can still use Travis for list linting, but the badge has no value in the readme. ✔️
Inspired by awesome-foo
orInspired by the Awesome project
kinda link at the top of the readme. The Awesome badge is enough. ✔️Go to the top and read it again.