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.

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

31 tests #37

Merged
merged 6 commits into from
Oct 16, 2022
Merged

31 tests #37

merged 6 commits into from
Oct 16, 2022

Conversation

skx
Copy link
Owner

@skx skx commented Oct 16, 2022

This pull-request, once complete, will restore our test-coverage back to 100%

  • builtin-package
    • We need to add tests for (file:lines..)
    • We need to add tests for (file:stat ..)
    • We need to add tests for (shell ..)
  • eval-package
    • We need a proper test of the timeout behaviour.
  • I should add my "alphabetical function order" perl-script to the repository/CI process too.

Nothing else appears to be outstanding now.

skx added 3 commits October 16, 2022 06:35
* Added file:lines.
* Updated help to call a lisp stdlib function.
@skx
Copy link
Owner Author

skx commented Oct 16, 2022

(rebased to add the alias functionality from #39)

skx added 3 commits October 16, 2022 07:36
This should be stable and robust, even in the CI process.
We rely upon /bin/echo, and the error-result of executing a
command that doesn't exist.
@skx skx merged commit 70004d3 into master Oct 16, 2022
@skx skx deleted the 31-tests branch October 16, 2022 05:59
@skx skx mentioned this pull request Oct 16, 2022
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.

1 participant