Delete twitter links to XSF account only #1489
Conversation
|
#1490 has 1 more commit on top of this one that goes further |
guusdk
left a comment
guusdk
left a comment
There was a problem hiding this comment.
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.
ralphm
left a comment
ralphm
left a comment
There was a problem hiding this comment.
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'
moparisthebest
force-pushed
the
rm-twitter
branch
from
7f78d1e to
4d11d23
Compare
moparisthebest
changed the title
|
Relevant changes were done in 5e2198a I'm closing this this PR, since it alters historical content, which is not intended (as stated above). |
4 participants
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.