Skip to content

Feature package manager support #4

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

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

lewandowskit93
Copy link

Added Swift Package Manager support & fixed codecov badge

@lewandowskit93 lewandowskit93 requested a review from lukkas January 25, 2020 00:56
@lewandowskit93 lewandowskit93 force-pushed the feature_package_manager_support branch from 3c37e0f to bb6a78f Compare January 25, 2020 00:57
@lewandowskit93 lewandowskit93 force-pushed the feature_package_manager_support branch from bb6a78f to 8564a4f Compare January 25, 2020 00:58
@lewandowskit93 lewandowskit93 force-pushed the feature_package_manager_support branch from 4d013b5 to dfaf02e Compare January 25, 2020 01:14
README.md Outdated

To install Visually using **Swift Package Manager** go through following steps:

1. Add following package dependency in you **Package.swift** ``` .package(url: "https://github.com/lukkas/Visually.git", from: "2.0.0") ```
Copy link
Owner

Choose a reason for hiding this comment

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

I think I'd just leave point 1. (and of course remove the number) and maybe add some context here, like so

dependencies: [
         .package(url: "https://github.com/lukkas/Visually.git", from: "2.0.0")
]

I'd leave out point 2 and the example. It is just information that you can find anywhere and I wouldn't like to focus on this in README too much

Copy link
Author

Choose a reason for hiding this comment

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

The "from" must be specified for a package dependency, I suggest changing "2.0.0" to -> "VERSION_NUMBER_HERE"
what do you think?

Copy link
Owner

Choose a reason for hiding this comment

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

I meant the entire second point of the instruction, not the version number. I don't want to get too specific of how to use SPM here. I think that up to 5 lines, including snippets is enough.

Copy link
Author

Choose a reason for hiding this comment

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

Sure, I'm onto it :)

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