Fix supported OS attributes for

Authored by vkrause on Aug 15 2019, 4:30 PM.



Move the "maintainer needed" to a note, so it's not showing up globally
for all modules. Also, change "macOS" back to "MacOSX" (despite technically
being the right name), as every other module still uses MacOSX and thus
using anything else essentially adds a new OS on, globally.

Diff Detail

R169 Kirigami
No Linters Available
No Unit Test Coverage
Build Status
Buildable 15151
Build 15169: arc lint + arc unit
vkrause created this revision.Aug 15 2019, 4:30 PM
Restricted Application added a project: Kirigami. · View Herald TranscriptAug 15 2019, 4:30 PM
Restricted Application added a subscriber: plasma-devel. · View Herald Transcript
vkrause requested review of this revision.Aug 15 2019, 4:30 PM
apol added a subscriber: apol.Aug 18 2019, 12:30 AM
apol added inline comments.

Isn't it called macOS nowadays? Or is it a keyword we have that we'd need to change elsewhere?

vkrause added inline comments.Aug 18 2019, 9:28 AM

See commit message, yes. We'd need to change it everywhere in order to show up as a single "macOS" entry on Alternatively we implement some form of OS normalization/mapping in kapidox.

IOW, this change improves the OS list from the current "Android | FreeBSD | Linux | MacOSX | Windows | iOS (maintainer needed) | macOS (maintainer needed)" to "Android | FreeBSD | Linux | MacOSX | Windows | iOS". Getting it to the ultimately correct "Android | FreeBSD | Linux | macOS | Windows | iOS" is a further change.

apol accepted this revision.Aug 18 2019, 10:48 PM
This revision is now accepted and ready to land.Aug 18 2019, 10:48 PM
This revision was automatically updated to reflect the committed changes.