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 README.md #39

Closed
wants to merge 1 commit into from
Closed

Update README.md #39

wants to merge 1 commit into from

Conversation

intelliot
Copy link

No description provided.

@ceeK
Copy link
Owner

ceeK commented Feb 15, 2021

Thanks @intelliot, I've now incorporated your changes into the README. Apologies that I didn't merge this PR directly; this was mainly due to your second example:

Use `Date()` for the sunrise and sunset for today

This is done via a default argument in the Swift initialiser, so isn't necessary in this example. I.e. Solar(coordinate: CLLocationCoordinate2D(latitude: 51.528308, longitude: -0.1340267)) does the same thing.

@ceeK ceeK closed this Feb 15, 2021
@intelliot intelliot deleted the patch-1 branch February 16, 2021 01:59
@1ec5 1ec5 mentioned this pull request Feb 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants