8000 pkg/nanocbor: Update for fixed nanocbor_skip_simple() [backport 2023.10] by bergzand · Pull Request #19988 · RIOT-OS/RIOT · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

pkg/nanocbor: Update for fixed nanocbor_skip_simple() [backport 2023.10] #19988

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

bergzand
Copy link
Member

backport of #19975 and #19983

Contribution description

Manually created the backport this time 😢

Includes both #19975 and #19983 to include the nanocbor bug fix for the nanocbor_skip_simple() function

Testing procedure

The usual nanocbor test in tests/pkg/nanocbor

Issues/PRs references

None

Important changes:
- Add stream-like interface for encoder
- Separate functions for number of items left in arrays and maps

(cherry picked from commit a250dfe)
Important changes:
- Fixed a bug in nanocbor_skip_simple with (b|t)str types
@bergzand bergzand added Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors) Area: pkg Area: External package ports Process: release backport Integration Process: The PR is a release backport of a change previously provided to master labels Oct 18, 2023
@benpicco benpicco added the CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR label Oct 18, 2023
@miri64
Copy link
Member
miri64 commented Oct 18, 2023

bors merge

@bors
Copy link
Contributor
bors bot commented Oct 18, 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.

@riot-ci
Copy link
riot-ci commented Oct 18, 2023

Murdock results

✔️ PASSED

b475907 pkg/nanocbor: Update for fixed nanocbor_skip_simple()

Success Failures Total Runtime
7937 0 7937 17m:10s

Artifacts

@miri64 miri64 added CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR and removed CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR labels Oct 19, 2023
@miri64
Copy link
Member
miri64 commented Oct 19, 2023

bors merge

@bors
Copy link
Contributor
bors bot commented Oct 19, 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.

@MrKevinWeiss
Copy link
Contributor

bors merge

@bors
Copy link
Contributor
bors bot commented Oct 23, 2023

Build succeeded!

The publicly hosted instance of bors-ng is deprecated and will go away soon.

If you want to self-host your own instance, instructions are here.
For more help, visit the forum.

If you want to switch to GitHub's built-in merge queue, visit their help page.

@bors bors bot merged commit 354e1ac into RIOT-OS:2023.10-branch Oct 23, 2023
@MrKevinWeiss MrKevinWeiss added this to the Release 2023.10 milestone Nov 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: pkg Area: External package ports CI: ready for build If set, CI server will compile all applications for all available boards for the labeled PR Process: release backport Integration Process: The PR is a release backport of a change previously provided to master Type: bug The issue reports a bug / The PR fixes a bug (including spelling errors)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants
0