Skip to content

docs: add deprecation section#16520

Merged
techdocsmith merged 7 commits intoapache:30.0.0from
317brian:add-depr-blurb
May 31, 2024
Merged

docs: add deprecation section#16520
techdocsmith merged 7 commits intoapache:30.0.0from
317brian:add-depr-blurb

Conversation

@317brian
Copy link
Contributor

@317brian 317brian commented May 30, 2024

Add deprecation section at the top of the release notes.

Once the migration guide for MVD -> array is merged, we can add a link to it.

@ektravel do you think we should add it to the upgrade notes too?

@317brian 317brian marked this pull request as ready for review May 30, 2024 21:13
Copy link
Contributor

@ektravel ektravel left a comment

Choose a reason for hiding this comment

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

Having two consecutive headings with the word "deprecations" can be confusing. Also calling them "upcoming" is misleading since the features are already deprecated. We should consider different headings to clearly differentiate the two sections. For example:
Upcoming removals
Important features, changes, and deprecations

@ektravel
Copy link
Contributor

@317brian I think we should add this section to the Upgrade notes topic, upgrade-notes.md. I don't think we need to repeat it in the Upgrade notes section of the Release notes topic, release-notes.md.

techdocsmith and others added 3 commits May 31, 2024 14:05
Co-authored-by: Katya Macedo  <38017980+ektravel@users.noreply.github.com>
Co-authored-by: Katya Macedo  <38017980+ektravel@users.noreply.github.com>
Co-authored-by: Katya Macedo  <38017980+ektravel@users.noreply.github.com>
Copy link
Contributor

@techdocsmith techdocsmith left a comment

Choose a reason for hiding this comment

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

LGTM

@317brian
Copy link
Contributor Author

Copy/pasted the section into upgrade-notes too:
image

@techdocsmith techdocsmith merged commit 3a32261 into apache:30.0.0 May 31, 2024
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.

3 participants