Skip to content

scripts: drop downstream 'west patch' command #736

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
merged 1 commit into from
Jan 27, 2025
Merged

Conversation

mniestroj
Copy link
Collaborator

Zephyr upstream has its own implementation of west patch subcommand,
which will basically allow the same kind of functionality.

Drop downstream west patch, since it is not used currently by the SDK.
That way there won't be any conflict after bumping Zephyr version.

Zephyr upstream has its own implementation of 'west patch' subcommand,
which will basically allow the same kind of functionality.

Drop downstream 'west patch', since it is not used currently by the SDK.
That way there won't be any conflict after bumping Zephyr version.

Signed-off-by: Marcin Niestroj <[email protected]>
Copy link

Visit the preview URL for this PR (updated for commit 24bfd94):

https://golioth-firmware-sdk-doxygen-dev--pr736-drop-west-patc-75sgv3ve.web.app

(expires Fri, 31 Jan 2025 10:32:35 GMT)

🔥 via Firebase Hosting GitHub Action 🌎

Sign: a9993e61697a3983f3479e468bcb0b616f9a0578

Copy link

Code Coverage

Code Coverage

Package Line Rate Branch Rate Health
include.golioth 75% 50%
port.linux 62% 35%
port.utils 58% 46%
port.zephyr 58% 25%
src 68% 30%
Summary 67% (2720 / 4086) 30% (1127 / 3712)

Copy link

codecov bot commented Jan 24, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

see 3 files with indirect coverage changes

@mniestroj mniestroj merged commit 88ff1db into main Jan 27, 2025
135 of 140 checks passed
@mniestroj mniestroj deleted the drop-west-patch branch January 27, 2025 11:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants