ci: install busted to run lua tests #1205
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Unfortunatly this does not mean that all tests passed. |
|
Maybe we should apply jeremybobbin/vis-test@db8210d and drop busted altogether. Alternativly, if we want to keep busted, there is fischerling@dab1f4a to fix the test failure detection. |
|
On Tue Sep 10, 2024 at 10:25 AM CEST, fischerling wrote:
Maybe we should apply jeremybobbin/vis-test@db8210d and drop busted altogether.
I am all for it, less dependencies better.
Best,
Matěj
…--
http://matej.ceplovi.cz/blog/, @***@***.***
GPG Finger: 3C76 A027 CA45 AD70 98B5 BC1D 7920 5802 880B C9D8
“Goodbye,” said the fox. “And now here is my secret, a very
simple secret: It is only with the heart that one can see
rightly; what is essential is invisible to the eye.”
-- Antoine de Saint-Exupery
|
|
The reason I didn't apply it was because Jeremy said he wasn't entirely happy with it. I'm not really seeing any particular issue with it. I'll confirm on the mailing list. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Install busted in the CI pipelines to run the included lua tests.