Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Make config-checker system easier to run. #413

Closed
verdammelt opened this issue Jun 6, 2021 · 1 comment · Fixed by #451
Closed

Make config-checker system easier to run. #413

verdammelt opened this issue Jun 6, 2021 · 1 comment · Fixed by #451
Labels
x:action/improve Improve existing functionality/content x:type/ci Work on Continuous Integration (e.g. GitHub Actions workflows)

Comments

@verdammelt
Copy link
Member

  1. add a test-op which runs the checks
  2. check-config should signal error if errors found
  3. ci-check-config should no longer be necessary.
@verdammelt verdammelt added x:type/ci Work on Continuous Integration (e.g. GitHub Actions workflows) x:action/improve Improve existing functionality/content labels Jun 11, 2021
@verdammelt
Copy link
Member Author

Also the output could be improved - such as sorting the list of concepts/exercises in the error messages.

Signalling an error is useful for CI - but can be annoying in the REPL as there are no useful restarts - I usually just want to run the command and see the results.

@verdammelt verdammelt mentioned this issue Aug 29, 2021
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
x:action/improve Improve existing functionality/content x:type/ci Work on Continuous Integration (e.g. GitHub Actions workflows)
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Adding test-op to config-checker system exercism/common-lisp
1 participant
@verdammelt