out_azure_blob: document path templating#2302
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
✅ Files skipped from review due to trivial changes (1)
📝 WalkthroughWalkthroughThe Azure Blob output documentation now documents path templating: the Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Suggested labels
Suggested reviewers
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
|
@nberlee rebase to pull in the changes that now conflict with your branch and we can review. |
|
I will put this on draft, there is not a reason for review at this time as the feature is still not merged. |
(refs fluent/fluent-bit#11178) Signed-off-by: Nico Berlee <nico.berlee@on2it.net>
|
@eschabell feature is merged this morning, and I rebased. Ready to be reviewed |
eschabell
left a comment
There was a problem hiding this comment.
The vale issue can be solved but is not crucial, merging!
Summary by CodeRabbit