voytech.m

Results 8 comments of voytech.m

MessageHeaders.getId already expects UUID so removing forced conversion seems hard. There is already SqsMessageHeaders class with some sqs specific headers support ( convenient getters ). Maybe adding support via sqs-id...

I have forked Your repository and created reporting using hiccup. I have also made some other improvements (in my optinion). Below short list of what I have done: - Created...

Yes so, HTML report task is using tmpdir and file which is incrementally growing with issues while different checkers reports issues in separate tasks. I think it should be separate...

Here is how it looks like (just in terms of HTML report support) 1. Each checker has been modified so that it returns vector of issue records (bikeshed and kibit...

Of course I am ready to make those changes. In fact I now feel much more convinced towards the idea of getting rid of separate report task and adding option...

@tolitius That is all doable and reasonable for me ;). But one question yet needs to be answered ... If You prefer to use multimethods (to enable custom reporters) for...

@tolitius I think a can make a PR. I have intoduced in my judgement very minimal set of changes: - added some functions in boot_check.clj - to allow persisting of...

@dakrone I have created PR for this. My only concern is that the bikeshed function **return type has changed**. So I do not know - maybe we should worry about...