Current `box-tests-report.tsv` has a problem havin...
# python-contributors
s
Current
box-tests-report.tsv
has a problem having it in the repo: it contains unstable values like execution time and sometimes paths to temporary files with random names (in stacktraces). So we are risking to have many changed lines in each PR... Anyway, it doesn't seem very critical to me
true story 1
p
Yeah, goos point , I was aware of it when implementing it. We can disable adding columns with timings and make paths relative. It should make the diff much cleaner 👍
s
JFTR: paths look like
/tmp/box.testsProject_pythonTest_2313166678231674774/testRootGW41/unitTest_kt42932/js-tests/JS_TESTS/kt42932.kt
, the
2313166678231674774
part is random one
p
Ah, in tmp and pseudorandom/with timestamps. In theory this can be addressed as well :)
Remove varying parts from box tests report by krzema12 · Pull Request #29 · krzema12/kotlin-python - should do the job. Once it's pushed, would be nice to rebase your changes on top of this change.
👌 1
s
Cool! "Before" and "after" are mixed up, btw 😅
p
oops, thx 😄 fixed
FTR, ready for review, finally green in CI
s
Thanks. I have some more tasks than usual now, so I guess I will return on Friday
✅ 1
👍 1