allow arbitrary platform strings in the @Available directive
#463
Conversation
rdar://104224866
|
@swift-ci Please test |
| if platform.rawValue.lowercased() == rawValue.lowercased() { | ||
| self = platform | ||
| return | ||
| } | ||
| } | ||
| return nil | ||
| if rawValue == "*" { |
There was a problem hiding this comment.
With this implementation as well, we can mimic Deprecated behavior. it would be so confusing to have @available() with Deprecated information passed to it.
There was a problem hiding this comment.
Do you mean allowing @Available(isDeprecated: true) once that is implemented? That will need to be handled as part of that implementation.
If you mean allowing @Available(MyPackage, isDeprecated, true), that should come for free without any extra work. I could add a test for that if you would like.
There was a problem hiding this comment.
I meant the first part, i.e @available(isDeprecated: true). sorry my response was never posted due to my internet connectivity.
There was a problem hiding this comment.
Right, deprecation status isn't available yet; see #441 for information.
|
@swift-ci Please test |
Bug/issue #, if applicable: rdar://104224866
Summary
In #440, the
@Availabledirective was limited to four platforms:iOS,macOS,watchOS,tvOS. This PR opens up that directive to take arbitrary platform strings, which will be passed along to the rendered page.Dependencies
None
Testing
Use the following metadata for testing:
Steps:
Sources/SwiftDocC/SwiftDocC.docc/SwiftDocC/Directives.md.bin/preview-docsChecklist
Make sure you check off the following items. If they cannot be completed, provide a reason.
./bin/testscript and it succeeded