My first Fluent Bit code PR ready for review: <htt...
# fluent-bit
e
My first Fluent Bit code PR ready for review: https://github.com/fluent/fluent-bit/pull/11192
🙌 2
p
Looks good, frustrating those were not flagged in review - that's what AI is for! 🙂 Can you backport as well?
e
To what?
p
4.0 and 4.1 ideally, should be an easy cherrypick
e
Would that be in the same PR?
I’ve not done “back porting” here before… guidance?
p
No, PRs are per branch so you have to create another for each backport.
e
That I can do….
p
Copy code
git checkout 4.0
git pull
git checkout -b backport
<copy changes>
git commit && git push
<create PR but ensure to target 4.0 base>
e
yep, that’s what I was gonna ask… that workflow….
p
repeat for 4.1
e
Perfect. Will do but not today.
p
make sure when you create the PR it does not target
master
(you can change it after)
e
Or let me see how fast I can do one….
Yes, good point on the PR target
p
it depends on what changes are also in the same file for
master
vs the others, otherwise you can checkout the files from your current branch:
Copy code
git checkout <PR branch> -- <file>
e
Whoops, my fork only has master
p
The PR titles are wrong but I can fix that and merge
e
Oh, didn’t know that was how you do the titles,thanks!