-
Notifications
You must be signed in to change notification settings - Fork 116
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
Fix a few minor warnings #449
Fix a few minor warnings #449
Conversation
506e110
to
73b03c6
Compare
Since the filter has no effect the second `count > 1` check is redundant
73b03c6
to
a8f5886
Compare
// Fill in the appropriate values here when that's fixed (https://github.com/apple/swift-docc/issues/441) | ||
return nil | ||
} | ||
// FIXME: Deprecated/Beta markings need platform versions to display properly in Swift-DocC-Render (rdar://56897597) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@QuietMisdreavus have I understood correctly that this FIXME applies generally (and should be kept) or did it only imply when "introduced" was nil
(which could never happen) in which case the FIXME can be removed?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think the comment still applies, since this is where the information will need to go when that's fixed.
@swift-ci please test |
@swift-ci please test |
@swift-ci please test |
Bug/issue #, if applicable:
Summary
This fixes a few minor warnings in the code:
sha512Checksum
was renamed tochecksum
)nil
Dependencies
n/a
Testing
n/a
Checklist
Make sure you check off the following items. If they cannot be completed, provide a reason.
./bin/test
script and it succeeded