Skip to content

deploy deprecation notice #1972

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 3 commits into from
Apr 15, 2025
Merged

deploy deprecation notice #1972

merged 3 commits into from
Apr 15, 2025

Conversation

mbostock
Copy link
Member

@mbostock mbostock commented Apr 15, 2025

@mbostock mbostock requested a review from tophtucker April 15, 2025 18:45
Copy link
Contributor

@tophtucker tophtucker left a comment

Choose a reason for hiding this comment

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

👍 looks good!

34314CBC-641C-46F4-99B1-202EAA1B9A03-22166-000C532885D357F2 5438963D-8681-4B4E-ADB8-BDBAEC5CBFBB-22166-000C5326EB08D2CD

@tophtucker
Copy link
Contributor

tophtucker commented Apr 15, 2025

zed autodetects the period as part of the URL, so command-clicking it goes here, which redirects to main release notes. annoying! 🤷 it's prob obvious enough visually, people can figure it out…

image

@mbostock
Copy link
Member Author

I’ll tweak the text to avoid the period confusion.

@mbostock mbostock merged commit 1475651 into main Apr 15, 2025
4 checks passed
@mbostock mbostock deleted the mbostock/deprecate-deploy branch April 15, 2025 21:15
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