-
Notifications
You must be signed in to change notification settings - Fork 19
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
Complete Open Source Checklist #23
Comments
@gautamarora do we have a particular asset to use for the Condé Nast Technology logo? |
yes, i will add it to templates and report back. |
@gautamarora @johnkpaul @pgoldrbx how do we feel about the we can remove / flatten a few those merge commits if we'd like, but is there any other feedback for what should be changed? |
@taveras - I think that is more relevant from the POV of not having commits that show sensitive data (before open sourcing) in the repo, but this project has been clean from the get go, so not an issue for it. |
The reason why that was asked for by the lawyers was because it helped mitigate any worry around accidentally revealing anything we didn't want to reveal when considering moving something from internal and private to public and open source. In this particular case, it was written as standalone from the beginning and I don't have concerns the git history could cause problems. |
👌 this makes sense, and understood! checking off that item on the list |
This is a master issue to work through the Condé Nast Open Source Checklist.
Documentation
Development
README Checklist
Badges Checklist:
The text was updated successfully, but these errors were encountered: