search

xvzcf / tls-interop-runner

Interoperability testing of TLS implementations.
Other
10 stars 13 forks source link

Moved code from cmd/ to internal/ and refactoring. #40

Closed xvzcf closed 3 years ago

xvzcf commented 3 years ago

There's certainly more to be done (like adding file name and line numbers to errors, which would allow for sort of a function trace, and adding documentation), but I decided to leave it at this since I have what I need for a CI runner and I'd like to get the CI setup by the end of the week hopefully.

xvzcf commented 3 years ago

It looks like this rips out the CLI. The restructure looks good, but I would suggest recreating a CLI. If we merge as is then there will be a period of time where the tests aren't runnable.

Oh no, the CLI should be unchanged, and all the make targets should still work.

cjpatton commented 3 years ago

Oops, you're right.