-
-
Notifications
You must be signed in to change notification settings - Fork 51
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
beyondcompare-np:Add version 5.x.y.z #385
base: master
Are you sure you want to change the base?
beyondcompare-np:Add version 5.x.y.z #385
Conversation
All changes look good. Wait for review from human collaborators. beyondcompare-np
|
44df9cc
to
f75ad98
Compare
2c9adcd
to
bc5c2bb
Compare
/verify |
All changes look good. Wait for review from human collaborators. beyondcompare-np
|
4d43a24
to
3e6982f
Compare
- commit suggestion, review finding Co-authored-by: HUMORCE <[email protected]>
- remove registry entries left using the installer in 'non-portable' mode
/verify |
All changes look good. Wait for review from human collaborators. beyondcompare-np
|
- address review finding Co-authored-by: Hsiao-nan Cheung <[email protected]>
- address review findings - ScoopInstaller#385 (comment) - ScoopInstaller#385 (comment) - ScoopInstaller#385 (comment) Co-authored-by: Hsiao-nan Cheung <[email protected]>
49cbbfb
to
b132c40
Compare
- revert 5dce899 - use uninstaller - use 'reg delete' recursively to remove installation entry showing up in 'Apps & features' otherwise
/verify |
All changes look good. Wait for review from human collaborators. beyondcompare-np
|
The context menu addition should be done via its installer instead of an extra script/registry file, for non-portable apps. unless it cannot. |
Add a non-portable installation of Beyond Compare version 5.x.y.z:
Relates to ScoopInstaller/Extras#13663
Relates to PR ScoopInstaller/Extras#14026
see discussion beyondcompare: Add version 5.x.y.z Extras#14026 (comment)
see discussion beyondcompare: Add version 5.x.y.z Extras#14026 (comment)
I have read the Contributing Guide.
FYI @niheaven, @HUMORCE