Skip to content
This repository has been archived by the owner on Nov 11, 2024. It is now read-only.

Commit

Permalink
Test
Browse files Browse the repository at this point in the history
Github action fix only: fail on AStyle failure.

The Github action wasn't failing if AStyle returned an error.  It does now.
  • Loading branch information
RobMeades committed Oct 22, 2023
1 parent 11f9f77 commit 6b5d66d
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 8 deletions.
3 changes: 1 addition & 2 deletions .github/workflows/style.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,5 +18,4 @@ jobs:
sudo apt install doxygen astyle python3
- name: check
run: |
python3 astyle.py
UBX_WORKDIR="$(pwd)" doxygen
python3 astyle.py && UBX_WORKDIR="$(pwd)" doxygen
6 changes: 3 additions & 3 deletions astyle.py
Original file line number Diff line number Diff line change
Expand Up @@ -13,11 +13,11 @@
stdout=subprocess.PIPE, stderr=subprocess.PIPE,
universal_newlines=True) as astyle:
fail = False
output = astyle.communicate()
for line in output:
output, _ = astyle.communicate()
for line in output.splitlines():
if line.startswith("Formatted"):
fail = True
print (line, end="")
print (line)
if astyle.returncode != 0:
fail = True
sys.exit(1 if fail else 0)
3 changes: 0 additions & 3 deletions gnss/api/u_gnss_cfg.h
Original file line number Diff line number Diff line change
Expand Up @@ -696,9 +696,6 @@ int32_t uGnssCfgValDelList(uDeviceHandle_t gnssHandle,
* values to delete; only the keyId member of each
* item is relevant, the others are ignored; must
* be NULL if numValues is 0. Wild-cards are permitted.
* @param numValues the number of items in the array pointed-to by pList;
* may be zero if the only purpose of this call is to
* execute a transaction.
* @param transaction use #U_GNSS_CFG_VAL_TRANSACTION_NONE to delete a single
* list of values; if you wish to begin deleting a sequence
* of values (which can each be single values or lists) that
Expand Down

0 comments on commit 6b5d66d

Please sign in to comment.