raspberry-pi-imager: Fix registry path - #18545
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
Included review availability: Your plan includes up to 8 reviews per rolling hour; 7 remain after this review. 📝 WalkthroughWalkthroughThe Scoop manifest updates the Windows telemetry opt-out registry path from Estimated code review effort: 1 (Trivial) | ~2 minutes Merge Risk: 🔵 Low · up to The change updates the Windows registry path used for the telemetry opt-out command. If the path does not match the application’s settings key, the opt-out may not take effect; merge is reasonable with explicit owner confirmation of the packaged release’s registry contract. 🚥 Pre-merge checks | ✅ 4 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (4 passed)
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 |
|
/verify |
|
All changes look good. Wait for review from human collaborators. raspberry-pi-imager
|
Due to an upstream ApplicationName change (raspberrypi/rpi-imager@3b4052a), the 2.x registry path differs from 1.x. #16988 specifically fixed this. The existing registry path is already correct.
|
|
Ah alright, didn't know it was changed, should've ran a blame first. |

I'm unsure of whether or not I made a mistake when I wrote this in the past, or if the actual registry path changed.
<manifest-name[@version]|chore>: <general summary of the pull request>