rust-secure-code / wg

Coordination repository for the Secure Code Working Group
151 stars 10 forks source link

Created a list of projects and added the link to the README. #32

Closed DevQps closed 5 years ago

DevQps commented 5 years ago

Description

What do you guys think about this? I wasn't sure about the categories, so I highly suspect that some naming might be improved.

closes #30

vakaras commented 5 years ago

Does anyone see any reason why this should not be merged? If not, I would suggest merging this pull request.

Shnatsel commented 5 years ago

I'm sorry about the delay. I know how frustrating and demotivating long PR review times can be, and it's a shame I have inadvertedly caused that here :disappointed:

The list looks good to me. However, I believe it would be better suited to a separate repository than a file under WG repo so that it can have a license, its own issue tracker, etc. So here's the course of action I propose:

  1. Split the list into a new repository, under your name
  2. Transfer the repository into the rust-secure-code organization
  3. Strip this PR into just an addition of a link to README

Due to legal restrictions I cannot create new repositories personally. So if you can split this off into a repo I'll accept the ownership transfer on the WG organization and land this PR immediately afterwards.

And once again, apologies for the delay.

DevQps commented 5 years ago

@Shnatsel It says I can't transfer it because You don’t have the permission to create repositories on rust-secure-code. Can you help me with this? The project I created is: https://github.com/DevQps/rustsec-projects

Shnatsel commented 5 years ago

That's weird, github docs do not mention anything like that. Perhaps you could transfer it to my account? I should have sufficient permissions to do pretty much anything with the org.

DevQps commented 5 years ago

Transfered it to you now!

Shnatsel commented 5 years ago

Aaand done! https://github.com/rust-secure-code/rustsec-projects

@DevQps you should still have direct commit access to it.

DevQps commented 5 years ago

Thanks man! I will edit this MR now.

Shnatsel commented 5 years ago

Merged. Thanks!