Skip to content

doc: releases: clean up entry order #93290

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

Conversation

danieldegrasse
Copy link
Contributor

@danieldegrasse danieldegrasse commented Jul 17, 2025

Clean up order of entries in the migration guide and release notes to be alphabetical. Combine two duplicate regulator sections, and move driver sections mistakenly placed under networking where they ought to be.

Also, add zephyr-keep-sorted entries to migration guide and release notes to keep alphabetical order enforced.

Migration guide rendered from this PR: https://builds.zephyrproject.io/zephyr/pr/93290/docs/releases/migration-guide-4.2.html

@zephyrbot zephyrbot added the Release Notes To be mentioned in the release notes label Jul 17, 2025
@zephyrbot zephyrbot requested review from dkalowsk and kartben July 17, 2025 22:35
dkalowsk
dkalowsk previously approved these changes Jul 17, 2025
@danieldegrasse danieldegrasse requested a review from a team July 17, 2025 23:59
stephanosio
stephanosio previously approved these changes Jul 18, 2025
jukkar
jukkar previously approved these changes Jul 18, 2025
@pdgendt
Copy link
Contributor

pdgendt commented Jul 18, 2025

This can be forced with

.. zephyr-keep-sorted-start re(^\w)

Title
=====

.. zephyr-keep-sorted-stop

Only lines that start with a word are taken into account.

@pdgendt
Copy link
Contributor

pdgendt commented Jul 18, 2025

You'd still find some unsorted entries 😅:

  • DAI < Devicetree
  • DMA < Devicetree (because CAPS)
  • Eeprom < Entropy
  • Enhanced Serial Peripheral Interface (eSPI) < Entropy
  • qSPI because it starts with lowercase goes to end of the list
  • SPI < Sensors (because CAPS)

Sorting case insensitive would require some more effort

@danieldegrasse
Copy link
Contributor Author

You'd still find some unsorted entries 😅:

* `DAI` < `Devicetree`

* `DMA` < `Devicetree` (because CAPS)

* `Eeprom` < `Entropy`

* `Enhanced Serial Peripheral Interface (eSPI)` < `Entropy`

* `qSPI` because it starts with lowercase goes to end of the list

* `SPI` < `Sensors` (because CAPS)

Sorting case insensitive would require some more effort

Great idea, thank you. Going to update PR with this, so that this gets pulled into future migration guides as well :)

@danieldegrasse danieldegrasse dismissed stale reviews from jukkar, stephanosio, and dkalowsk via e1b9873 July 18, 2025 15:03
@danieldegrasse danieldegrasse force-pushed the doc/release-notes-cleanup branch from 0119c58 to e1b9873 Compare July 18, 2025 15:03
@danieldegrasse danieldegrasse changed the title doc: releases: migration-guide: clean up entry order doc: releases: clean up entry order Jul 18, 2025
@danieldegrasse danieldegrasse force-pushed the doc/release-notes-cleanup branch from e1b9873 to d6b4741 Compare July 18, 2025 15:12
@danieldegrasse
Copy link
Contributor Author

Great idea, thank you. Going to update PR with this, so that this gets pulled into future migration guides as well :)

Should be updated now- added the same sorting to release notes, where it caught several things to sort

@@ -164,6 +162,24 @@ Display
the format expected by Zephyr. This change ensures proper behavior of both
display and video capture samples.

Eeprom
Copy link
Member

Choose a reason for hiding this comment

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

This should be all caps since it is an acronym for "electrically erasable programmable read-only memory"

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed

Clean up order of entries in the migration guide to be alphabetical.
Combine two duplicate regulator sections, and move driver sections
mistakenly placed under networking where they ought to be.

Signed-off-by: Daniel DeGrasse <[email protected]>
Clean up order of entries to be alphabetically sorted in the "new apis"
section

Signed-off-by: Daniel DeGrasse <[email protected]>
Enforce sorting of entries, using zephyr-keep-sorted. This will make
sure that entries don't get out of order again within the release notes
or migration guide by throwing a compliance error.

Note that autogenerated areas do not have this enforcement enabled.

Signed-off-by: Daniel DeGrasse <[email protected]>
@danieldegrasse danieldegrasse force-pushed the doc/release-notes-cleanup branch from d6b4741 to 53ce308 Compare July 18, 2025 16:01
Copy link

@dkalowsk dkalowsk merged commit 1ba9504 into zephyrproject-rtos:main Jul 18, 2025
20 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Release Notes To be mentioned in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants