Skip to content

Conversation

@pdgendt
Copy link
Contributor

@pdgendt pdgendt commented May 16, 2025

  • Add release notes for v1.3.0 and v1.4.0
  • Updated the API for v1.4.0

@pdgendt pdgendt mentioned this pull request May 16, 2025
20 tasks
@pdgendt pdgendt force-pushed the west-docs branch 2 times, most recently from 59b8928 to 28f1934 Compare May 16, 2025 07:28
pdgendt added 2 commits May 16, 2025 09:28
Add release notes for v1.3.0 and v1.4.0.

Signed-off-by: Pieter De Gendt <pieter.degendt@basalte.be>
Update the west API documentation for the added active_only arguments
to as_dict/as_frozen_dict/as_yaml/as_frozen_yaml.

Signed-off-by: Pieter De Gendt <pieter.degendt@basalte.be>
@sonarqubecloud
Copy link

@pdgendt pdgendt requested review from carlescufi, kartben and marc-hb May 16, 2025 14:00
@pdgendt pdgendt added the DNM This PR should not be merged (Do Not Merge) label May 16, 2025
@pdgendt pdgendt marked this pull request as ready for review May 16, 2025 14:09
@github-actions github-actions bot requested review from nashif and swinslow May 16, 2025 14:10
Copy link
Contributor

@marc-hb marc-hb left a comment

Choose a reason for hiding this comment

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

The content is fine but do we really need to keep duplicating this to the wrong git repo? I wonder why the 1.3.0 release notes were missed! (not)

Moving the entire West documentation is a challenge (discussed in zephyrproject-rtos/west#799).

On the other hand, merely replacing the duplicate at
https://docs.zephyrproject.org/latest/develop/west/release-notes.html with a pointer to the original at https://github.com/zephyrproject-rtos/west/releases sounds like a small de-duplication effort and I don't think the extra click will bother any user. On the contrary, users will enjoy the de-duplication, the up-to-date notes and the better hypertext.

@pdgendt
Copy link
Contributor Author

pdgendt commented May 16, 2025

The content is fine but do we really need to keep duplicating this to the wrong git repo? I wonder why the 1.3.0 release notes were missed! (not)

I agree, but didn't want to change the process for this minor release.

Moving the entire West documentation is a challenge (discussed in zephyrproject-rtos/west#799).

I still think it would be better to have a dedicated documentation site for west, but this would be something for west v2.

On the other hand, merely replacing the duplicate at docs.zephyrproject.org/latest/develop/west/release-notes.html with a pointer to the original at zephyrproject-rtos/west/releases sounds like a small de-duplication effort and I don't think the extra click will bother any user. On the contrary, users will enjoy the de-duplication, the up-to-date notes and the better hypertext.

This was somewhat discussed with @mbolivar in discord, where I suggest to move large chunks and keep the Zephyr specifics in the Zephyr documentation.

@marc-hb
Copy link
Contributor

marc-hb commented May 16, 2025

I agree, but didn't want to change the process for this minor release.

How much of a "process" change would this de-duplication be? It looks trivial to me but I might miss stuff.

@pdgendt
Copy link
Contributor Author

pdgendt commented May 17, 2025

How much of a "process" change would this de-duplication be? It looks trivial to me but I might miss stuff.

Probably trivial, but the effort is already done, I don't see an issue with doing this still for v1.4.0. I'll work on this after the release if that's okay.

@pdgendt pdgendt removed the DNM This PR should not be merged (Do Not Merge) label May 22, 2025
@carlescufi carlescufi merged commit f8a4b0a into zephyrproject-rtos:main May 22, 2025
26 of 27 checks passed
@pdgendt pdgendt deleted the west-docs branch May 22, 2025 14:56
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.

4 participants