Skip to content
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

Closed
wants to merge 0 commits into from

Conversation

hummingbird-automation[bot]
Copy link
Contributor

##
##===----------------------------------------------------------------------===##
##===----------------------------------------------------------------------===##
##
## This source file is part of the SwiftNIO open source project
Copy link
Contributor

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?

Copy link
Member

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

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I see, thanks!

##
##===----------------------------------------------------------------------===##
##===----------------------------------------------------------------------===##
##
## This source file is part of the SwiftNIO open source project
##
## Copyright (c) 2017-2019 Apple Inc. and the SwiftNIO project authors
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same here?

@thoven87
Copy link
Contributor

Can the verify documentation step failed with libs that swift jobs is not dependent on?

@adam-fowler
Copy link
Member

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

Copy link
Contributor

@thoven87 thoven87 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants