Few improvements

[TAdev0]

• issue #
• follows contribution guide
• code change includes tests
• breaking change

@b-j-roberts i will close my other PR , i had conflict issues i coudlnt solve easily and it was faster for me to do this way

[TAdev0]

@b-j-roberts I also replaced alll assert! with assert() function in the codebase. While its nice in tests as you can easily print long strings, using the macro actually costs way more steps because of the ByteArray construct

[b-j-roberts]

@b-j-roberts I also replaced alll assert! with assert() function in the codebase. While its nice in tests as you can easily print long strings, using the macro actually costs way more steps because of the ByteArray construct

Nice catch, had slipped my mind.