search

JBenda / inkcpp

Inkle Ink C++ Runtime with JSON>Binary Compiler
MIT License
70 stars 13 forks source link

Ink Proof Badge Generation #24

Closed brwarner closed 3 years ago

brwarner commented 3 years ago

Trying to get Github CI to generate one badge per operating system that prints the inkproof test results.

brwarner commented 3 years ago

Not sure why the cp command with the * is failing on Windows. The ls command right before it says there's something in the directory... Probably missing something.

JBenda commented 3 years ago

D:\a\inkcpp\inkcpp/build/bin/* I think this is the Problem, because Windows uses \ instead of / you try to copy the file "inkcpp/build/bin/*" in directory D:\a\inkcpp

brwarner commented 3 years ago

I have it set to use the bash shell though. Hrm. I'll give it a shot.

github-actions[bot] commented 3 years ago

Reporting from ubuntu-20.04. Ink Proof Result: / .

github-actions[bot] commented 3 years ago

Reporting from macos. Ink Proof Result: 0 / 96.

github-actions[bot] commented 3 years ago

Reporting from ubuntu. Ink Proof Result: 42 / 96.

github-actions[bot] commented 3 years ago

Reporting from windows. Ink Proof Result: 0 / 96.

brwarner commented 3 years ago

So this runs now but I think the badge action I'm using is busted. The urls lead to server errors. Also would be nice if all the bot comments were merged into one. I'll improve soon.

JBenda commented 3 years ago

Cool that it's start working :)

but why only 96 inkproofs? they should be a little over 100.

brwarner commented 3 years ago

Implemented this better using the sticky comment action in another branch.