Pavlos Rontidis

Results 222 comments of Pavlos Rontidis

Also, please see https://github.com/vectordotdev/vector/blob/ac1e975d109e4ab575e2acb2f3766aa8084c8ea3/docs/DOCUMENTING.md?plain=1#L89

> @pront I don't have the file in my PR, do you want me to open a new PR or just add the file in my existing PR? https://github.com/vectordotdev/vector/blob/master/.github/workflows/semantic.yml#L182 Feel...

Hello, we don't have the capacity to get to this right now. We always welcome PRs and we do our best to review them ASAP. > Regardless, it does seem...

> Please use DNS TTL as "the" TTL. Sounds right 👍 I suppose this concept will come up during the PR review. @johnhtodd you are welcome to help review this...

Can you please run: ``` ./website/scripts/cue.sh fmt CI=true make check-docs ``` and commit the result? I tried to do it but cannot push to the fork directly.

> Hey @pront, I just raised the PR because I like the github diff, I'm still testing the change and the integration tests on this branch don't work yet. Sounds...

> This PR includes the changes in #22033 Let's try to merge this https://github.com/vectordotdev/vector/pull/22033

> [![CLA assistant check](https://camo.githubusercontent.com/9f3ba258263b002ca0462e8569342be921d4fd0be994b92e17f8aa0fb7bea1a3/68747470733a2f2f636c612e64617461646f6768712e636f6d2f70756c6c2f62616467652f6e6f745f7369676e6564)](https://cla.datadoghq.com/vectordotdev/vector?pullRequest=21272) Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our [Contributor License Agreement](https://cla.datadoghq.com/vectordotdev/vector?pullRequest=21272) before we can...

@bruceg would you like to take another look before we merge this? There was a force push and multiple updates after you last reviewed.