Skip to content

reduce image size#42

Merged
keiravillekode merged 3 commits intoexercism:mainfrom
keiravillekode:smaller-image
May 4, 2026
Merged

reduce image size#42
keiravillekode merged 3 commits intoexercism:mainfrom
keiravillekode:smaller-image

Conversation

@keiravillekode
Copy link
Copy Markdown
Contributor

The image reduces from 524MB to 286MB

The image reduces from 524MB to 286MB
@keiravillekode keiravillekode requested a review from a team as a code owner May 3, 2026 19:50
Comment thread bin/process_results.awk
@@ -0,0 +1,87 @@
function jesc(s) {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
function jesc(s) {
#!/bin/awk -f
function jesc(s) {

Comment thread bin/process_results.awk Outdated
BEGIN {
last_test = ""
while ((getline line < test_src) > 0) {
while (match(line, /RUN_TEST\([A-Za-z_][A-Za-z0-9_]*\)/)) {
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
while (match(line, /RUN_TEST\([A-Za-z_][A-Za-z0-9_]*\)/)) {
while (line ~ /RUN_TEST\([A-Za-z_][A-Za-z0-9_]*\)/) {

Comment thread bin/process_results.awk Outdated
completed { next }
/^[^:]+_test\.c:[0-9]+:[A-Za-z_][A-Za-z0-9_]*:(PASS|FAIL)(:.*)?$/ {
line = $0
p1 = index(line, ":")
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you want the split() function?

@keiravillekode keiravillekode merged commit b9c6363 into exercism:main May 4, 2026
1 check passed
@keiravillekode keiravillekode deleted the smaller-image branch May 4, 2026 00:24
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