-
-
Notifications
You must be signed in to change notification settings - Fork 1
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
Update from Hummingbird Project Template #32
Conversation
scripts/validate.sh
Outdated
## | ||
##===----------------------------------------------------------------------===## | ||
##===----------------------------------------------------------------------===## | ||
## | ||
## This source file is part of the SwiftNIO open source project |
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.
Should this text be updated?
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.
We need to include the original NIO text, as that is what it was when I copied the file across. I've added a Hummingbird header because the file has been altered since then
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 see, thanks!
scripts/validate.sh
Outdated
## | ||
##===----------------------------------------------------------------------===## | ||
##===----------------------------------------------------------------------===## | ||
## | ||
## This source file is part of the SwiftNIO open source project | ||
## | ||
## Copyright (c) 2017-2019 Apple Inc. and the SwiftNIO project authors |
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.
Same here?
Can the verify documentation step failed with libs that swift jobs is not dependent on? |
Yes, we need to get to a state where all docs are happy. The issue is docs can reference other packages so I have to build all docs at the same time. We were happy, but then swift 6 got released and swift 6 is super fussy about the function headers and now we have a load of errors which we need to fix in released builds. I have most of them fixed just haven't done releases |
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.
lgtm
d857225
to
0412c4e
Compare
Automated update of files from https://github.com/hummingbird-project/hummingbird-project-template