0ex / more-awesome

An extensive list of "awesome" lists to help you find resources and starting points on every topic.
GNU Affero General Public License v3.0
67 stars 6 forks source link

Add Photonics #380

Closed 0exbot closed 1 year ago

0exbot commented 1 year ago

Pull request from @joamatab.

add photonics resources https://github.com/joamatab/awesome_photonics

Add curated list For photonic engineers, physicists and hobbyists

[ boilerplate snipped ]

0exbot commented 1 year ago

@-joamatab on 2022-02-17 06:45 says: unicorn

0exbot commented 1 year ago

@-sindresorhus on 2022-02-17 07:24 says: 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.

0exbot commented 1 year ago

@-Karneades on 2022-02-17 13:29 says: With the purpose of helping, went again over your repo and PR. One of the things is that every contributor should also review others. You could do it for my malware persistence list ;) and at least for another open PR.

0exbot commented 1 year ago

@-inoue0426 on 2022-02-22 20:10 says: You need to review at least two other PR and then write down the repository you reviewed like this, sindresorhus/awesome#-2034.

0exbot commented 1 year ago

@-joamatab on 2022-03-01 05:37 says: sounds good

I've reviewed https://github.com/sindresorhus/awesome/pull/2079 and https://github.com/sindresorhus/awesome/pull/2084

0exbot commented 1 year ago

@-Karneades on 2022-03-01 07:13 says:

sounds good

I've reviewed sindresorhus/awesome#-2079 and sindresorhus/awesome#-2084

What exactly did you review? :) You should leave a useful comment for improvement in open pull request. 2079 was already merged and closed in January and in 2084 you didn't comment at all :D You could do it for my malware persistence list PR ;) and at least for another open PR.

Regarding your PR: I commented multiple times on your PR, on the first one, and this one, and still various things are missing from the guidelines. Please check other PRs like this, so sindresorhus has less work himself.

0exbot commented 1 year ago

@-sindresorhus on 2022-06-05 10:10 says: Closing as the list is very far from being mergable and there's been no activity for a long time.