Unit testing
#2190
Replies: 2 comments
-
What about small scale PRs which at minimum require Lua unit testing would it be preferable they also need unit testing in C++? I think it's a good idea and it would be good practice for a project of this scale. I had a quick search and busted looks good. |
Beta Was this translation helpful? Give feedback.
0 replies
-
Somebody found the idea to do it! Totally agree with it. It can solve some of compatibility problem when somebody make a PR. But I have no ideas how to implement it 🤔 . |
Beta Was this translation helpful? Give feedback.
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
I propose to require some kind of unit tests for new PRs.
The main blocker of merging PRs fast right now (IMHO) is the lack of trust, and the fear of breaking something.
Properly written unit tests can filter these errors quite effectively.
What do you think?
Which library should we use?
How should we deal with Lua code? Use a resource (Maybe along a Lua unit test library?)?
Beta Was this translation helpful? Give feedback.
All reactions