Skip to content
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

docs: Remove release notes and fully capture them in GitHub Releases #3380

Merged
merged 4 commits into from
Mar 29, 2024

Conversation

binste
Copy link
Contributor

@binste binste commented Mar 23, 2024

Closes #3339. I added the missing releases to GitHub Releases and made sure that all content from the docs is captured in there.

"Release Notes" in the navigation bar now links directly to GitHub:
image

And the old /releases/changes.html URL redirects to GitHub after showing a message that the content got moved:

image

I moved the draft release notes for 5.3.0 into RELEASING.md so we don't loose them. I'll remove them when I'll do the 5.3 release. To capture release notes in the future, see Joel's suggestion here

…e everything from changes.rst. Added missing versions in GitHub Releases page so that all information is captured there.
@binste
Copy link
Contributor Author

binste commented Mar 23, 2024

@jonmmease https://github.com/altair-viz/altair/actions/runs/8402900588/job/23012797705?pr=3380 fails due to some different outputs from VegaFusion. It's unrelated to this PR as I'm only modifying the docs here. Maybe something to do with the latest VF release? Would you mind taking a look? Otherwise, I can look into it later this week.

@jonmmease
Copy link
Contributor

Yes, it's from a change in the latest VegaFusion release. I fixed the tests in #3377, which I think is ready to merge as soon as someone has time to look over it.

Copy link
Contributor

@joelostblom joelostblom left a comment

Choose a reason for hiding this comment

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

These changes look good to me and works when building locally. Thanks for taking the time to move all the old releases to GitHub!

--------------------------
- Update Vega-Lite from version 5.16.3 to version 5.17.0;
see `Vega-Lite Release Notes <https://github.com/vega/vega-lite/releases>`_.
We have moved the release notes to GitHub. You'll be automatically redirected there in 3 seconds, or you can click `here <https://github.com/altair-viz/altair/releases>`_.
Copy link
Contributor

Choose a reason for hiding this comment

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

Is there an urgent need to redirect automatically?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

No urgent need. I thought redirecting in general is good to not lose users. An additional click is always an opportunity for someone to drop out. And 3 seconds felt like enough time for me to read the message but no strong preference on how we handle this. What do you think?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Just stumbled on pandas doing the same with around 1 second of wait time: https://pandas.pydata.org/pandas-docs/stable/generated/pandas.DataFrame.to_json.html

Copy link
Contributor

Choose a reason for hiding this comment

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

Double negative alert: I thought not to auto redirect in order to not lose users..

At least we have the same intention😀!

Copy link
Contributor

Choose a reason for hiding this comment

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

I don't notice a redirect on the pandas website, it stays within the pandas documentation page isn't it?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ah I think I see what you mean! Same intentions but I think not redirecting might indeed be a better way to achieve this. I'll remove the automated redirect so users can decide themselves if they want to go to GitHub or stay in the Altair docs.

Re the Pandas docs: Maybe they changed that... Yesterday it showed briefly a message before redirecting. Now it directly goes to stable/reference instead of stable/generated. Anyway :)

@binste binste merged commit d09b7de into vega:main Mar 29, 2024
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Only track release notes in one place
4 participants