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

Swift PM and Linux Support #35

Closed
wants to merge 5 commits into from
Closed

Conversation

cpageler93
Copy link

Hi,

I'd like to participate in this project by adding support for Swift Package Manager and some fixes for Linux os.

Please let me know when i missed something.

Thanks

Package.resolved Outdated
@@ -0,0 +1,16 @@
{
Copy link
Contributor

Choose a reason for hiding this comment

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

@cpageler93 I guess maybe we don't want to include package.resolved in this package. The document says it seems like having Package.resolved as a dependent has no effect to package which depends on it

https://github.com/apple/swift-package-manager/blob/master/Documentation/Usage.md#resolved-versions-packageresolved-file

but still it's kinda odd to ping the version here as this is a lib package, it makes no sense to pin the version of embassy here

Copy link
Contributor

@fangpenlin fangpenlin left a comment

Choose a reason for hiding this comment

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

Overall looks good, just the Package.resolved, I think we don't need it here

@cpageler93
Copy link
Author

I moved the PR to #36 because i made a little mess with commit 363da44 in this PR

@cpageler93 cpageler93 closed this Jul 6, 2018
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.

None yet

2 participants