-
Notifications
You must be signed in to change notification settings - Fork 881
Review issues and PRs for concerning content #650
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
Comments
What exactly does this mean? |
I think what Jen had in mind was deleting/removing closed issues when we open source the repo. I don't know if that's a possible or something I think we should do but she asked me to look into it. |
It appears that admins can permanently delete issues. I am not sure if we want to do this since issues may contain historical information and there is no clear benefit to doing this. I think the only thing we want to do is make sure there is no sensitive information mentioned on GitHub but keeping issues should be fine. I'll let @misskniss @kylecarbs weigh in before closing. |
Waiting on response from others before moving forward on this. |
Sounds like we don't want to delete closed issues but instead we want to check for any concerning content that shouldn't be public. |
@bpmct is going to look through PRs and post his list as well. |
The following list of PRs either mention sensitive information or a Notion/Shortcut link: (there are some mentions of coder/m or cdr/m not added here. they do not mention sensitive IP, just implementations we are porting over)
|
I edited descriptions to remove notion links from descriptions. A few cases that reference a notion link in a comment, but I am not concerned with those. |
@tjcran did you edit my list too? I still see some sensitive information. Also, ICYMI we need to edit these and then delete the revisions (otherwise that information will be visible when we open source). |
Description
Research if it's possible to close closed issues (delete them) so we start with a fresh state.
The text was updated successfully, but these errors were encountered: