Nikolai Luthman
Nikolai Luthman
UPDATE: I've made some changes, so the config/run parameters are not valid anymore. I'll update it when I get back from vacation. I've created a patch in my own fork...
@vivekratnavel cool! I'll look at it ASAP, so in a week or two, since I'm deep into something else atm. UPDATE: Just wanted to let you know that this might...
Thanks for the reminder! I'll have some time right over new years to merge with current and finish up, and make a pull request and then we'll see how it...
> @nuth Let me know if you need some help with the testing or point me to the branch it's on. Yes, that'd be very nice, since I seem to...
> I also have a similar use case with multiple sacred databases. Is this feature being actively worked on? What remaining tasks need to be handled before merging? I could...
The solution seems to follow 12 factors (https://12factor.net/) more or less, so it should be configurable through an environment variable, which is a good idea anyways when running things in...
Indeed, it's not perfect, but in my mind hard coding the config file would be less in line with 12 factor than a config file controlled by an environment variable......
> Was this code passing the web tests on your branch? I'm finding that e.g. in runsTable.test.js the expects around mockAxios want a leading '/' like '/api/...'. Does this make...
UPDATE: I see somebody already did this... I've been writing my own client in java (lost the code, but still). I was thinking of splitting up the file client side...
Hmm, from what I can tell you can already adjust the severity for each rule https://github.com/detekt/detekt/issues/3274 I've also tested this and it seems to work? I also see the point...