-
-
Notifications
You must be signed in to change notification settings - Fork 27.6k
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 IT Quotes #903
Add IT Quotes #903
Conversation
Please read the requirements above - this can be reopened in 30 days and when all above points are met. |
@sindresorhus Sorry about other PR, but I created it because to adjust commit message. |
It hasn't been around for 30 days yet for a start |
@davisonio Can you reopen now?? |
@@ -328,6 +328,7 @@ Check out my [blog](https://blog.sindresorhus.com) and follow me on [Twitter](ht | |||
- [Fantasy](https://github.com/RichardLitt/awesome-fantasy) | |||
- [Podcasts](https://github.com/guipdutra/awesome-geek-podcasts) | |||
- [Email Newsletters](https://github.com/vredniy/awesome-newsletters) | |||
- [IT Quotes](https://github.com/victorlaerte/awesome-it-quotes) |
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 entry in the Awesome list should include a short description about the list project/theme (not the list itself)
|
The badge should be on the right side of the title. Drop the Awesome image. It's only useful if you actually integrate the list title with the Awesome logo, like https://github.com/vuejs/awesome-vue The list tagline should be in normal font size and not bold. Contribution guidelines should be in a separate |
readme.md
Outdated
@@ -328,6 +328,7 @@ Check out my [blog](https://blog.sindresorhus.com) and follow me on [Twitter](ht | |||
- [Fantasy](https://github.com/RichardLitt/awesome-fantasy) | |||
- [Podcasts](https://github.com/guipdutra/awesome-geek-podcasts) | |||
- [Email Newsletters](https://github.com/vredniy/awesome-newsletters) | |||
- [IT Quotes](https://github.com/victorlaerte/awesome-it-quotes) - Best IT quotes over the history. |
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.
Needs a better description. And Best
is already implied. This is an awesome list.
All topics of the first comment has been done, but the following one I could not understand.
Quotes ?or table header? |
That's because you removed it in victorlaerte/awesome-it-quotes@5355bed#diff-04c6e90faac2675aa89e2176d2eec7d8 You still need a tagline/intro under the list heading. For example: https://github.com/jbmoelker/progressive-enhancement-resources |
Is it ok now ? |
This reverts commit 899af7e.
https://github.com/victorlaerte/awesome-it-quotes
This is an list of awesome IT quotes. The aims is to get together all relevant quotes said over the history of IT. There's no much data yet, but I'm still selecting new ones.
By submitting this pull request I confirm I've read and complied with the below requirements.
Failure to properly do so will just result in the pull request being closed and everyone's time wasted. Please read it twice. Most people miss many things.
Add Name of List
, notUpdate readme.md
orAdd awesome list
.Contents
as the first section,A
/An
prefixes in the descriptions.Node.js
, notNodeJS
ornode.js
.