Fotis Koutoulakis

Results 16 comments of Fotis Koutoulakis

Hi @dtolnay, pinging you here, as evaluating this one for going in would be an incredible help to a lot of workflows (at least for me). PR looks clean, has...

Hi @dtolnay, many thanks for a very prompt response. In light of this decision, what would you suggest is the best way to construct a `CxxVector` and pass it to...

Hello @lopopolo . I am aware of a Windows-specific requirement for this issue - I am sadly not in the position to check it because I lack access to any...

Hello, don't know how useful my review on this would be, but happy to do a review if needed. I do have a question though, just to build my understanding...

On second thoughts, I had a look at the diff, and this appears to be just a sync PR, needing just a stamp. Happy to give it to unblock us.

> Very nice! Could you update the docs for tools in init/action.yml too? Yes, this is my plan before the end of the ticket as a whole. I was originally...

Apologies, I had to force push a `rebase` because of some issues with the `CHANGELOG`. What has changed in 5a08657: * I moved the logging to the downstream function `getCodeQLSource`,...

Hi @henrymercer, this is now ready for another round of reviews.

Hello, I have spent some time on this and I have been unable to reproduce this (more precisely: _I'm seeing the expected behaviour_) following the instructions. My extension version is...

Follow up: Downloaded a version of the bundle (`v2.16.4`) and pointed the extension to its local path. I have also checked out the tag `codeql-cli/v2.16.4` of the `github/codeql` repository and...