std-max

Results 19 comments of std-max

LGTM too. I would even say that this should not be a generic settings as it should be the default behaviour. Maybe for v5.0? But for the next minor release,...

I was browsing through packages and notice the same issue, nice to see it is already opened 😉 I've some questions: - What does the `p` mean after 2008 or...

@LarsAsplund In fact I added a test that calls `test_runner_setup` just because it was done in a previous test like the one at line 318 (`banner("Should loop over enabled_test_case once...

Done @LarsAsplund :)

No joke, I literally fell asleep while waiting for the checks to pass You can now merge @LarsAsplund haha

I think the problem is due to the print of the content of the JSON object. This content contains a lot of NUL characters. https://github.com/VUnit/vunit/blob/c23790d8159e0ced8cd8ffd00c50d6d2c5425e3d/examples/vhdl/json4vhdl/src/test/tb_json_gens.vhd#L29 In the example, we should...

@eine maybe you have an idea to solve this?

@umarcor I will propose a PR this afternoon for the erroneous example. I like the solution you proposed with a separate log file. Maybe I (or you if you have...

Apparently a check failed but @LarsAsplund you have already fixed it in #835

CI looks good after rebasing :)