Skip to content

Update 2022-01-12-this-week-in-rust.md (#1) #2790

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

Merged
merged 5 commits into from Jan 13, 2022
Merged

Update 2022-01-12-this-week-in-rust.md (#1) #2790

merged 5 commits into from Jan 13, 2022

Conversation

ghost
Copy link

@ghost ghost commented Jan 11, 2022

No description provided.

oliver added 3 commits January 10, 2022 23:16
* Update 2022-01-12-this-week-in-rust.md

* Update 2022-01-12-this-week-in-rust.md

* Update 2022-01-12-this-week-in-rust.md

* Update 2022-01-12-this-week-in-rust.md

* Update 2022-01-12-this-week-in-rust.md

* Update 2022-01-12-this-week-in-rust.md

* Update 2022-01-12-this-week-in-rust.md
Copy link
Contributor

@ericseppanen ericseppanen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why does this change the title of an existing link?

Unless there's a compelling reason, I think the link text should match the article title.

@ghost
Copy link
Author

ghost commented Jan 12, 2022

Yes, the project name has been updated, technically references to the legacy name are outdated.

I think that starting from the recent update, modifying all current/future references makes sense, even if the blog content its self hasn't been modified. From the perspective a future reader it's the legacy name which would seem out of place.

@ericseppanen
Copy link
Contributor

Sorry, but the policy here is to have the link title match the original post. As long as the authors of the blog chose to leave their title alone, we will too.

@ericseppanen
Copy link
Contributor

ericseppanen commented Jan 12, 2022

Another note: the TWiR editors do collect links from most of the common sources. The OSX link and the Fyrox name change announcement are unique; the rest will be handled by #2794.

We do appreciate the submissions, but there's probably no need to submit things that are already broadcast widely on the big forums; we'll pick those up every week.

@nellshamrell
Copy link
Contributor

Hi @kw-fn!

I appreciate you collecting these links! As @ericseppanen mentioned, our policy is to keep the titles the same as the titles of the articles. You are most welcome to collect and submit these, though if they are distributed on Reddit or Hackernews we will likely pick them up.

Thank you for your interest in This Week in Rust!

Copy link
Contributor

@nellshamrell nellshamrell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ty!

@nellshamrell nellshamrell merged commit e3ef665 into rust-lang:master Jan 13, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants