Sotiris Nanopoulos

Results 31 comments of Sotiris Nanopoulos

See https://github.com/ysugimoto/falco/pull/102

I have this implemented just wanted to push it the upstream rather than having this patch on our fork. I believe this might be useful for other people who version...

That is soooooo dope, super excited for this PR. will review tomorrow

when this + https://github.com/ysugimoto/falco/pull/99 is merged I will pull it in to our internal branch and test it on our VCL corpus as well and report back if I run...

I think we can go in DFS style parsing where when we see an `include` statement we push it into the AST

I think this works. I am wondering if it would be easier (less process) if we just merge on main and then tag main: * every x PRs * every...

So for example would you be ok with me merging https://github.com/ysugimoto/falco/pull/89?

Thanks for reverting! We can fix it, sorry about that we dont use this pipeline so I completely forgot that we need to change this as well.

@thhous-msft / @anrossi might be able to chime in since they worked on cross platform event tracing for https://github.com/microsoft/msquic it would be awesome if we could get the event tracing...

huge +1 on this one