-
Notifications
You must be signed in to change notification settings - Fork 370
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
Delete twitter links to XSF account only #1489
base: master
Are you sure you want to change the base?
Conversation
#1490 has 1 more commit on top of this one that goes further |
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 board vote called for no removal of preexisting links to Tweets. This PR seems to remove a couple.
I'd not merge this PR because of that, but do one of the other provided options instead.
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.
My intent of the wording of the motion to not promote our account at X was just to remove generic references to it from the footer of the site, similar places like the press kit, and future post, news letters, and other materials that specifically mention our X account as one of our outlets. It was specifically not my intention to alter history in any way, or to remove links to individual posts (ours or others') or prevent those in the future.
rg -l twitter.com | xargs sed -i '/^. *.Twitter..https:..twitter\.com.xmpp./d' rg -l twitter.com | xargs sed -i '/^. *.Twitter.*https:..twitter\.com.xmpp./d' rg -l twitter.com | xargs sed -ri '/^. *.?Twitter.*https:..twitter\.com.xmpp.?/d'
7f78d1e
to
4d11d23
Compare
This change is per the board vote today.
First commit was done entirely with the script included in the commit message.
Second was done manually.