Skip to content

release-notes.txt: add 2023.04 release notes#19503

Merged
bors[bot] merged 1 commit intoRIOT-OS:masterfrom
jia200x:pr/2023.04/release_notes
Apr 27, 2023
Merged

release-notes.txt: add 2023.04 release notes#19503
bors[bot] merged 1 commit intoRIOT-OS:masterfrom
jia200x:pr/2023.04/release_notes

Conversation

@jia200x
Copy link
Member

@jia200x jia200x commented Apr 25, 2023

Contribution description

This are the release notes for the 2023.04 Release. Please give it a look and make sure that your work in the last 3 months is present here.

Testing procedure

  • Read the document

Issues/PRs references

@github-actions github-actions bot added the Area: doc Area: Documentation label Apr 25, 2023
@jia200x jia200x added Process: release cycle Integration Process: The PR is connected to the release cycle (e.g. release notes) Area: doc Area: Documentation and removed Area: doc Area: Documentation labels Apr 25, 2023
benpicco
benpicco reviewed Apr 25, 2023
View reviewed changes
release-notes.txt Outdated
addresses (#19258)
* gcoap: Finish the gcoap_get_resource_list_tl ->
gcoap_get_resource_list renaming (#19295)
* sys/flash_utils: helpers to store data in flash (#18148)
Copy link
Contributor

@benpicco benpicco Apr 25, 2023 •
edited
Loading

Choose a reason for hiding this comment

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

Not sure why this is an API change

But the new servo API and the ESP_WIFI_SSID -> WIFI_SSID change sure is.
vfs_umount() has gained a flag to for forced unmount.

Copy link
Member Author

@jia200x jia200x Apr 26, 2023

Choose a reason for hiding this comment

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

The original PR was tagged as API Change (#18148).
Is it intended? @maribu

Copy link
Member

@maribu maribu Apr 27, 2023

Choose a reason for hiding this comment

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

Yes, phydat_unit_to_str() is no longer thread safe on Harvard architectures and deprecated in favor of phydat_unit_print() and phydate_unit_write(), which are thread safe and, subjectively, easier to use.

It is a sublte API change likely nobody notices, as phydat_unit_to_str() was likely never used by applications directly, but rather indirect by including the saul shell command or by using phydat_to_json. Both have been updated to use the not-deprecated API. So I don't think that anyone would be affected.

Copy link
Member Author

@jia200x jia200x Apr 27, 2023

Choose a reason for hiding this comment

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

ok, I remove it from the API Changes then.

benpicco
benpicco reviewed Apr 25, 2023
View reviewed changes
miri64
miri64 reviewed Apr 25, 2023
View reviewed changes
chrysn
chrysn reviewed Apr 26, 2023
View reviewed changes
@MrKevinWeiss
Copy link
Contributor

MrKevinWeiss commented Apr 26, 2023

Should we add the boards that have been tested for this release (maybe just pass fail results and link the issue)?

gschorcht
gschorcht reviewed Apr 26, 2023
View reviewed changes
MrKevinWeiss
MrKevinWeiss requested changes Apr 26, 2023
View reviewed changes
aabadie
aabadie reviewed Apr 26, 2023
View reviewed changes
MrKevinWeiss
MrKevinWeiss previously approved these changes Apr 27, 2023
View reviewed changes
@MrKevinWeiss MrKevinWeiss dismissed their stale review April 27, 2023 08:53

All good but I shouldn't be the one to ack

@MrKevinWeiss MrKevinWeiss added the Process: needs >1 ACK Integration Process: This PR requires more than one ACK label Apr 27, 2023
@github-actions github-actions bot added the Process: missing approvals Integration Process: PR needs more ACKS (handled by action) label Apr 27, 2023
maribu
maribu approved these changes Apr 27, 2023
View reviewed changes
Copy link
Member

@maribu maribu left a comment

Choose a reason for hiding this comment

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

Looks good to me. Two nit-picks inline. Feel free to squash directly, or to ignore the nitpicks.

release-notes.txt Outdated
Comment on lines +153 to +157
+ drivers/mrf24j40: add support for IEEE 802.15.4 Radio HAL (#18472)
+ drivers/mtd_sdcard: add mtd_sdcard_default module (#19216)
+ makefiles/tools/serial.inc.mk: Allow detection of debug adapter (#19119)
+ pkg/lwext4: add lightweight implementation of the ext2/3/4 filesystem (#18682)
+ pkg/tinyusb: add GD32VF103 support (#19256)
Copy link
Member

@maribu maribu Apr 27, 2023

Choose a reason for hiding this comment

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

This don't look boards related; maybe they should be moved in a different category?

release-notes.txt Outdated
+ gh-actions: report release-tests result to Matrix (#19102)
* CI: .murdock: fix bors fast build (#19241)
* CI: .murdock: use build checkout for ccache tmp (#19274)
* CI: seperate check-labels and check-commits workflows (#19120)
Copy link
Member

@maribu maribu Apr 27, 2023

Choose a reason for hiding this comment

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

Suggested change
* CI: seperate check-labels and check-commits workflows (#19120)
* CI: separate check-labels and check-commits workflows (#19120)

release-notes.txt Outdated
addresses (#19258)
* gcoap: Finish the gcoap_get_resource_list_tl ->
gcoap_get_resource_list renaming (#19295)
* sys/flash_utils: helpers to store data in flash (#18148)
Copy link
Member

@maribu maribu Apr 27, 2023

Choose a reason for hiding this comment

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

Yes, phydat_unit_to_str() is no longer thread safe on Harvard architectures and deprecated in favor of phydat_unit_print() and phydate_unit_write(), which are thread safe and, subjectively, easier to use.

It is a sublte API change likely nobody notices, as phydat_unit_to_str() was likely never used by applications directly, but rather indirect by including the saul shell command or by using phydat_to_json. Both have been updated to use the not-deprecated API. So I don't think that anyone would be affected.

@jia200x
Copy link
Member Author

jia200x commented Apr 27, 2023

@maribu done!

MrKevinWeiss
MrKevinWeiss approved these changes Apr 27, 2023
View reviewed changes
Copy link
Contributor

@MrKevinWeiss MrKevinWeiss left a comment

Choose a reason for hiding this comment

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

ACK

@jia200x jia200x force-pushed the pr/2023.04/release_notes branch from 651f36d to 2c111ae Compare April 27, 2023 13:26
@github-actions github-actions bot removed the Process: missing approvals Integration Process: PR needs more ACKS (handled by action) label Apr 27, 2023
@MrKevinWeiss
Copy link
Contributor

MrKevinWeiss commented Apr 27, 2023

bors merge

@bors
Copy link
Contributor

bors bot commented Apr 27, 2023

🕐 Waiting for PR status (GitHub check) to be set, probably by CI. Bors will automatically try to run when all required PR statuses are set.

@benpicco benpicco added CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR CI: skip compile test If set, CI server will run only non-compile jobs, but no compile jobs or their dependent jobs labels Apr 27, 2023
@riot-ci
Copy link

riot-ci commented Apr 27, 2023

Murdock results

✔️ PASSED

2c111ae release-notes.txt: add 2023.04 release notes

Success Failures Total Runtime
1 0 1 48s

Artifacts

@MrKevinWeiss
Copy link
Contributor

MrKevinWeiss commented Apr 27, 2023

bors merge

bors bot added a commit that referenced this pull request Apr 27, 2023
@bors @jia200x
Merge #19503
5290e5f 
19503: release-notes.txt: add 2023.04 release notes r=MrKevinWeiss a=jia200x



Co-authored-by: Jose Alamos <jose@alamos.cc>
@MrKevinWeiss
Copy link
Contributor

MrKevinWeiss commented Apr 27, 2023

bors cancel

@bors
Copy link
Contributor

bors bot commented Apr 27, 2023

Canceled.

@MrKevinWeiss
Copy link
Contributor

MrKevinWeiss commented Apr 27, 2023

bors merge

@bors
Copy link
Contributor

bors bot commented Apr 27, 2023

Build succeeded:

@bors bors bot merged commit d371f13 into RIOT-OS:master Apr 27, 2023
@MrKevinWeiss MrKevinWeiss mentioned this pull request Apr 27, 2023
Merged
bors bot added a commit that referenced this pull request Apr 27, 2023
@bors @jia200x
Merge #19518
e6840ad 
19518: release-notes.txt: add 2023.04 release notes [backport 2023.04] r=MrKevinWeiss a=MrKevinWeiss

# Backport of #19503



Co-authored-by: Jose Alamos <jose@alamos.cc>
@benpicco benpicco added this to the Release 2023.07 milestone Aug 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@chrysn chrysn chrysn left review comments

@miri64 miri64 miri64 left review comments

@benpicco benpicco benpicco left review comments

@aabadie aabadie aabadie left review comments

@gschorcht gschorcht gschorcht left review comments

@maribu maribu maribu approved these changes

@MrKevinWeiss MrKevinWeiss MrKevinWeiss approved these changes

Assignees

No one assigned

Labels

Area: doc Area: Documentation CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR CI: skip compile test If set, CI server will run only non-compile jobs, but no compile jobs or their dependent jobs Process: needs >1 ACK Integration Process: This PR requires more than one ACK Process: release cycle Integration Process: The PR is connected to the release cycle (e.g. release notes)

Projects

None yet

Milestone

Release 2023.07

Development

Successfully merging this pull request may close these issues.

9 participants

@jia200x @MrKevinWeiss @riot-ci @chrysn @miri64 @benpicco @aabadie @maribu @gschorcht