Skip to content

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

Merged
binste merged 4 commits into
vega:mainfrom
binste:move_changelog
Mar 29, 2024
Merged

binste merged 4 commits into
vega:mainfrom
binste:move_changelog

Conversation

@binste

@binste binste commented Mar 23, 2024

Copy link
Copy Markdown
Contributor

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

binste added 2 commits March 23, 2024 16:33
…e everything from changes.rst. Added missing versions in GitHub Releases page so that all information is captured there.
@binste

binste commented Mar 23, 2024 •

Copy link
Copy Markdown
Contributor Author

@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
Copy Markdown
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.

@joelostblom joelostblom left a comment •

Copy link
Copy Markdown
Contributor

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!

Comment thread doc/releases/changes.rst Outdated
--------------------------
- 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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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
Copy Markdown
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 :)

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