docs: replace all instances of twitter.com with x.com#6355
docs: replace all instances of twitter.com with x.com#6355Eomm merged 1 commit intofastify:mainfrom cseas:chore-twitter-links
Conversation
jsumners
left a comment
There was a problem hiding this comment.
In my opinion, the links are still valid. I don't know of a single person that calls that site by the rename. If the links don't work, sure; but I bet they do.
Correct, the links are valid but they all redirect to x.com now. This is just a housekeeping PR to avoid the redirect. |
|
Maybe removing twitter/x is better? |
Yep, people can redirect to their social networks from their Github profile. |
@fastify/core thoughts? |
gurgunday
left a comment
There was a problem hiding this comment.
I'm neutral but let's merge this anyway while discussing it, twitter domain might be retired soon
Replaced outdated twitter links
Checklist
npm run test && npm run benchmark --if-presentand the Code of conduct