Skip to content

Omit -toplevel.rexx from solution#151

Open
keiravillekode wants to merge 1 commit into
exercism:mainfrom
keiravillekode:no-toplevel-in-solution
Open

Omit -toplevel.rexx from solution#151
keiravillekode wants to merge 1 commit into
exercism:mainfrom
keiravillekode:no-toplevel-in-solution

Conversation

@keiravillekode

@keiravillekode keiravillekode commented Apr 6, 2026

Copy link
Copy Markdown
Contributor

Each .meta/config.json solution previously mentioned a ...-toplevel.rexx file.

These are now omitted.

We also update each test entry to show ...-check.rexx

This change was generated using

find exercises/practice -name config.json -path  '*/.meta/*' -exec sh -c 'jq ".files.solution = [.files.solution[0]]" "$1" > "$1.tmp" && mv "$1.tmp" "$1"' _ {} \;

find exercises/practice -name config.json -path '*/.meta/*' -exec sh -c 'jq ".files.test = [.files.test[] | sub(\"^test-\"; \"\") + \"-check.rexx\"]" "$1" > "$1.tmp" && mv "$1.tmp" "$1"' _ {} \;

bin/configlet fmt -u

@keiravillekode

Copy link
Copy Markdown
Contributor Author

Note that we could omit the reformatting by instead running

sed -i 's/\("solution": \["\([^"]*\)\.rexx"\), "[^"]*-toplevel\.rexx"\]/\1]/' exercises/practice/*/.meta/config.json

but note that bin/configlet fmt -u reapplies the reformatting.

Each .meta/config.json solution previously mentioned a ...-toplevel.rexx file.

These are now omitted.

We also update each test entry to show `...-check.rexx`

This change was generated using

```bash
find exercises/practice -name config.json -path  '*/.meta/*' -exec sh -c 'jq ".files.solution = [.files.solution[0]]" "$1" > "$1.tmp" && mv "$1.tmp" "$1"' _ {} \;

find exercises/practice -name config.json -path '*/.meta/*' -exec sh -c 'jq ".files.test = [.files.test[] | sub(\"^test-\"; \"\") + \"-check.rexx\"]" "$1" > "$1.tmp" && mv "$1.tmp" "$1"' _ {} \;

bin/configlet fmt -u
```
@keiravillekode keiravillekode force-pushed the no-toplevel-in-solution branch from c7048a4 to 816ed66 Compare April 10, 2026 05:29
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