D8207: Use https for KDE urls

2017-10-08 Thread Kevin Funk
kfunk added a comment. In https://phabricator.kde.org/D8207#153510, @mpyne wrote: > I support the idea, but I think there's a few more spots where we can convert http:// to https://. I ran `ag --only-matching --no-heading 'http:.*k.*\.org'` in the kcoreaddons source directory, with the

D8207: Use https for KDE urls

2017-10-08 Thread Kevin Funk
This revision was automatically updated to reflect the committed changes. Closed by commit R244:3cc8b8f5ad26: Use https for KDE urls (authored by kfunk). CHANGED PRIOR TO COMMIT https://phabricator.kde.org/D8207?vs=20491&id=20502#toc REPOSITORY R244 KCoreAddons CHANGES SINCE LAST UPDATE ht

D8207: Use https for KDE urls

2017-10-08 Thread Aleix Pol Gonzalez
apol accepted this revision. apol added a comment. This revision is now accepted and ready to land. Mostly tests though ^^'. REPOSITORY R244 KCoreAddons BRANCH master REVISION DETAIL https://phabricator.kde.org/D8207 To: kfunk, apol Cc: apol, mpyne, #frameworks

D8207: Use https for KDE urls

2017-10-08 Thread Michael Pyne
mpyne added a comment. I support the idea, but I think there's a few more spots where we can convert http:// to https://. I ran `ag --only-matching --no-heading 'http:.*k.*\.org'` in the kcoreaddons source directory, with the current version of this patch already applied, and got the follow

D8207: Use https for KDE urls

2017-10-08 Thread Kevin Funk
kfunk created this revision. Restricted Application added a project: Frameworks. Restricted Application added a subscriber: Frameworks. REVISION SUMMARY Make sure links in dialogs point to https:// urls REPOSITORY R244 KCoreAddons BRANCH master REVISION DETAIL https://phabricator.kde.or