diff options
author | Christopher Haster <chaster@utexas.edu> | 2022-02-19 08:51:52 +0300 |
---|---|---|
committer | Christopher Haster <chaster@utexas.edu> | 2022-03-11 23:36:23 +0300 |
commit | b045436c23a7607a3cd808da0fe9279909fd16ea (patch) | |
tree | 59198742f60432d8b130bb547170000533c54596 /.github | |
parent | ead50807f1ca3fdf2da00b77a0ce02651ded2d13 (diff) |
Added size-sort options to scripts/code.py
Now with -s/--sort and -S/--reverse-sort for sorting the functions by
size.
You may wonder why add reverse-sort, since its utility doesn't seem
worth the cost to implement (these are just helper scripts after all),
the reason is that reverse-sort is quite useful on the command-line,
where scrollback may be truncated, and you only care about the larger
entries.
Outside of the command-line, normal sort is prefered.
Fortunately the difference is just the sign in the sort key.
Note this conflicts with the short --summary flag, so that has been
removed.
Diffstat (limited to '.github')
-rw-r--r-- | .github/workflows/release.yml | 12 | ||||
-rw-r--r-- | .github/workflows/test.yml | 4 |
2 files changed, 8 insertions, 8 deletions
diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml index a1a1a43..cf856c9 100644 --- a/.github/workflows/release.yml +++ b/.github/workflows/release.yml @@ -84,7 +84,7 @@ jobs: select(.context == "results / code").description | capture("Code size is (?<result>[0-9]+)").result' \ prev-results.json || echo 0)" - ./scripts/code.py -u results/code-thumb.csv -s | awk ' + ./scripts/code.py -u results/code-thumb.csv --summary | awk ' NR==2 {printf "Code size,%d B",$2} NR==2 && ENVIRON["PREV"]+0 != 0 { printf " (%+.1f%%)",100*($2-ENVIRON["PREV"])/ENVIRON["PREV"]} @@ -95,7 +95,7 @@ jobs: select(.context == "results / code (readonly)").description | capture("Code size is (?<result>[0-9]+)").result' \ prev-results.json || echo 0)" - ./scripts/code.py -u results/code-thumb-readonly.csv -s | awk ' + ./scripts/code.py -u results/code-thumb-readonly.csv --summary | awk ' NR==2 {printf "Code size<br/>(readonly),%d B",$2} NR==2 && ENVIRON["PREV"]+0 != 0 { printf " (%+.1f%%)",100*($2-ENVIRON["PREV"])/ENVIRON["PREV"]} @@ -106,7 +106,7 @@ jobs: select(.context == "results / code (threadsafe)").description | capture("Code size is (?<result>[0-9]+)").result' \ prev-results.json || echo 0)" - ./scripts/code.py -u results/code-thumb-threadsafe.csv -s | awk ' + ./scripts/code.py -u results/code-thumb-threadsafe.csv --summary | awk ' NR==2 {printf "Code size<br/>(threadsafe),%d B",$2} NR==2 && ENVIRON["PREV"]+0 != 0 { printf " (%+.1f%%)",100*($2-ENVIRON["PREV"])/ENVIRON["PREV"]} @@ -117,7 +117,7 @@ jobs: select(.context == "results / code (migrate)").description | capture("Code size is (?<result>[0-9]+)").result' \ prev-results.json || echo 0)" - ./scripts/code.py -u results/code-thumb-migrate.csv -s | awk ' + ./scripts/code.py -u results/code-thumb-migrate.csv --summary | awk ' NR==2 {printf "Code size<br/>(migrate),%d B",$2} NR==2 && ENVIRON["PREV"]+0 != 0 { printf " (%+.1f%%)",100*($2-ENVIRON["PREV"])/ENVIRON["PREV"]} @@ -128,7 +128,7 @@ jobs: select(.context == "results / code (error-asserts)").description | capture("Code size is (?<result>[0-9]+)").result' \ prev-results.json || echo 0)" - ./scripts/code.py -u results/code-thumb-error-asserts.csv -s | awk ' + ./scripts/code.py -u results/code-thumb-error-asserts.csv --summary | awk ' NR==2 {printf "Code size<br/>(error-asserts),%d B",$2} NR==2 && ENVIRON["PREV"]+0 != 0 { printf " (%+.1f%%)",100*($2-ENVIRON["PREV"])/ENVIRON["PREV"]} @@ -139,7 +139,7 @@ jobs: select(.context == "results / coverage").description | capture("Coverage is (?<result>[0-9\\.]+)").result' \ prev-results.json || echo 0)" - ./scripts/coverage.py -u results/coverage.csv -s | awk -F '[ /%]+' ' + ./scripts/coverage.py -u results/coverage.csv --summary | awk -F '[ /%]+' ' NR==2 {printf "Coverage,%.1f%% of %d lines",$4,$3} NR==2 && ENVIRON["PREV"]+0 != 0 { printf " (%+.1f%%)",$4-ENVIRON["PREV"]} diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml index 6d633f8..44dcf8b 100644 --- a/.github/workflows/test.yml +++ b/.github/workflows/test.yml @@ -229,7 +229,7 @@ jobs: | select(.context == env.CONTEXT).description | capture("Code size is (?<result>[0-9]+)").result' \ || echo 0)" - export DESCRIPTION="$(./scripts/code.py -u $f -s | awk ' + export DESCRIPTION="$(./scripts/code.py -u $f --summary | awk ' NR==2 {printf "Code size is %d B",$2} NR==2 && ENVIRON["PREV"]+0 != 0 { printf " (%+.1f%%)",100*($2-ENVIRON["PREV"])/ENVIRON["PREV"]}')" @@ -427,7 +427,7 @@ jobs: | capture("Coverage is (?<result>[0-9\\.]+)").result' \ || echo 0)" export DESCRIPTION="$( - ./scripts/coverage.py -u results/coverage.csv -s | awk -F '[ /%]+' ' + ./scripts/coverage.py -u results/coverage.csv --summary | awk -F '[ /%]+' ' NR==2 {printf "Coverage is %.1f%% of %d lines",$4,$3} NR==2 && ENVIRON["PREV"]+0 != 0 { printf " (%+.1f%%)",$4-ENVIRON["PREV"]}')" |