mrWheel / YAPP_Box

Yet Another Parametric Projectbox Box
https://willem.aandewiel.nl/index.php/2022/01/02/yet-another-parametric-projectbox-generator/
MIT License
276 stars 46 forks source link

Feature request: test CI/CD workflow for this project #63

Open DeflateAwning opened 10 months ago

DeflateAwning commented 10 months ago

When making upgrades, it would be helpful to be able to confirm that a list of example/sample test files still compile successfully and correctly (or at least very similarly to how they did before).

I'm not sure exactly what the solution looks like, but a GitHub Workflow with many test files for this project could be very helpful as it continues to grow, to continue that the project is working in each branch/PR before it's merged.

Does anyone involved in this project know of any other OpenSCAD library/library-like projects that have this sort of testing workflow setup? Otherwise, it can't imagine it would be that hard to setup ourselves.

rosenhauer commented 10 months ago

I'm a newby when it comes to GitHub. But I like the idea of having some sort of automated testing.

I've tried to include example/sample files with all of the recent changes. I normally have a sample of the prior function as a reference that I haven't broken anything.

On Thu, Jan 25, 2024 at 4:41 PM DeflateAwning @.***> wrote:

When making upgrades, it would be helpful to be able to confirm that a list of example/sample test files still compile successfully and correctly (or at least very similarly to how they did before).

I'm not sure exactly what the solution looks like, but a GitHub Workflow with many test files for this project could be very helpful as it continues to grow, to continue that the project is working in each branch/PR before it's merged.

Does anyone involved in this project know of any other OpenSCAD library/library-like projects that have this sort of testing workflow setup? Otherwise, it can't imagine it would be that hard to setup ourselves.

— Reply to this email directly, view it on GitHub https://github.com/mrWheel/YAPP_Box/issues/63, or unsubscribe https://github.com/notifications/unsubscribe-auth/AIBTUKBHFEJWYWGHUN5KMLDYQLNRFAVCNFSM6AAAAABCLI4LSSVHI2DSMVQWIX3LMV43ASLTON2WKOZSGEYDCMRYGA4DQMA . You are receiving this because you are subscribed to this thread.Message ID: @.***>

-- Dave