Commit Graph

10 Commits

Author SHA1 Message Date
Wyatt Hepler
3520967a78 pw_status: Remove update_style.py
Since the status style migration is complete, remove update_style.py
(rather than fix the missing Python dependency on pw_presubmit).

Change-Id: Ie003544be49a3ea7ce62287cd531d5154e4b0037
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/30606
Pigweed-Auto-Submit: Wyatt Hepler <hepler@google.com>
Commit-Queue: Auto-Submit <auto-submit@pigweed.google.com.iam.gserviceaccount.com>
Reviewed-by: Keir Mierle <keir@google.com>
2021-01-23 01:43:34 +00:00
Wyatt Hepler
b7302cb7f6 pw_status: Script updates; fix deprecation message
Change-Id: Ie6702f749d28b02c3640bf3a9c0faeb37b803aa3
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/29262
Pigweed-Auto-Submit: Wyatt Hepler <hepler@google.com>
Reviewed-by: Ewout van Bekkum <ewout@google.com>
Commit-Queue: Wyatt Hepler <hepler@google.com>
2021-01-11 23:30:31 +00:00
Wyatt Hepler
5a53dfd1af pw_status: Introduce pw::OkStatus()
Status::Ok() is too similar to Status::ok(); having both status.ok() and
status.Ok() compile but mean different things is confusing.

This CL introduces a pw::OkStatus() free function that is an alias for
pw::Status(). This is intended to be a concise, readable way to create
a status with code PW_STATUS_OK.

Change-Id: I85d2ea1d14662c6c4d455219cfeced90c5f92218
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/28900
Reviewed-by: Ewout van Bekkum <ewout@google.com>
Commit-Queue: Auto-Submit <auto-submit@pigweed.google.com.iam.gserviceaccount.com>
Pigweed-Auto-Submit: Wyatt Hepler <hepler@google.com>
2021-01-11 17:56:09 +00:00
Wyatt Hepler
d109398072 pw_status: Read/write bytes for update_style.py
Using bytes avoids potential Unicode decode errors.

Change-Id: I0f58a19c0f06147d19cc18499d44c819abdb8cd1
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/24480
Reviewed-by: Armando Montanez <amontanez@google.com>
Commit-Queue: Wyatt Hepler <hepler@google.com>
2020-11-13 20:41:27 +00:00
Wyatt Hepler
5083907255 pw_status: Functions for checking the status code
- Add IsErrorCode() functions to simplify checking for specific status
  codes.
- Automatically replace some cases of status == Status::Code() with
  status.IsCode() in update_style.py.

Change-Id: I13be834a2413d694d41a7755d2e317c35d34c5bf
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/19521
Reviewed-by: Keir Mierle <keir@google.com>
Reviewed-by: Ewout van Bekkum <ewout@google.com>
Commit-Queue: Wyatt Hepler <hepler@google.com>
2020-11-12 04:48:14 +00:00
Wyatt Hepler
3b4047da68 Python: Add py.typed files per PEP 561
- PEP 561 (https://www.python.org/dev/peps/pep-0561/) specifies that
  packages that wish to support type checking must provide a py.typed
  marker file. Add py.typed to all packages.
- Add missing __init__.py files to pw_cli and pw_hdlc_lite.

Change-Id: I32d652222e9909a5ac4a5612c2dd1683b18b2d36
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/22200
Reviewed-by: Alexei Frolov <frolv@google.com>
Commit-Queue: Wyatt Hepler <hepler@google.com>
2020-10-22 19:24:02 +00:00
Wyatt Hepler
bb57d9cdd5 pw_status: Docs update
- Expand status docs.
- Fix update_style.py docstring.

Change-Id: I44323ca293d801d9c8b5e24ee3353046ec3d5db2
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/19164
Reviewed-by: Ewout van Bekkum <ewout@google.com>
Reviewed-by: Alexei Frolov <frolv@google.com>
Commit-Queue: Wyatt Hepler <hepler@google.com>
2020-09-30 16:31:23 +00:00
Wyatt Hepler
de4bb2391a pw_status: Tool for migrating Status constants
update_style.py updates Status and StatusWithSize uses from Status::CODE
to Status::Code() style.

Change-Id: I81102b3e524c6a123845d2ee152310a7a2991f3d
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/19000
Reviewed-by: Keir Mierle <keir@google.com>
2020-09-25 23:08:32 +00:00
Wyatt Hepler
1748f65d0d pw_hdlc_lite: HdlcRpcClient class
Provide a class that configures a pw_rpc.Client for use over HDLC.

Change-Id: I191b9ef85926b11421c870f9d914b6a111dd8795
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/18461
Reviewed-by: Keir Mierle <keir@google.com>
Commit-Queue: Wyatt Hepler <hepler@google.com>
2020-09-18 23:32:33 +00:00
Wyatt Hepler
d3d1cb0755 pw_status: Python package
Change-Id: I1b3e9a9cf2d8c685771c4df97a14dc267177655f
Reviewed-on: https://pigweed-review.googlesource.com/c/pigweed/pigweed/+/13162
Reviewed-by: Alexei Frolov <frolv@google.com>
Commit-Queue: Wyatt Hepler <hepler@google.com>
2020-06-30 18:10:31 +00:00