Skip to content

Add "crates.io: Download changes" blog post #1269

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 1 commit into from
Mar 11, 2024

Conversation

Turbo87
Copy link
Member

@Turbo87 Turbo87 commented Mar 8, 2024

Rendered

Screen Shot 2024-03-08 at 11 29 30

r? @rust-lang/crates-io

@Turbo87 Turbo87 added the content An issue related to an article. label Mar 8, 2024
@Turbo87 Turbo87 force-pushed the crates-io-download-changes branch from 8870b3c to 60e78e5 Compare March 8, 2024 10:55
Copy link
Contributor

@LawnGnome LawnGnome left a comment

Choose a reason for hiding this comment

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

LGTM. I have a few non-blocking suggestions, but I'm also fine if this just gets published as is.

Copy link
Member

@carols10cents carols10cents left a comment

Choose a reason for hiding this comment

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

Some suggestions, up to you what to take if anything!

@carols10cents
Copy link
Member

Possible improvement to the image, feel free to take, reject, or reimplement as you wish:

download-graph with deployment point marked

@Turbo87 Turbo87 force-pushed the crates-io-download-changes branch from 60e78e5 to 02532ca Compare March 11, 2024 08:50
@Turbo87
Copy link
Member Author

Turbo87 commented Mar 11, 2024

@LawnGnome @carols10cents thanks for the feedback. I've integrated most of these changes :)

@Turbo87 Turbo87 merged commit 3f273a0 into rust-lang:master Mar 11, 2024
@Turbo87 Turbo87 deleted the crates-io-download-changes branch March 11, 2024 08:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content An issue related to an article.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants