Skip to content
This repository has been archived by the owner on May 20, 2021. It is now read-only.

API return objects are now iterable #38

Merged
merged 2 commits into from
Jan 14, 2020

Conversation

Vysybyl
Copy link
Contributor

@Vysybyl Vysybyl commented Jan 13, 2020

No description provided.

@Detrous
Copy link
Owner

Detrous commented Jan 13, 2020

Hi @Vysybyl . Thanks for all your changes. For this PR I need to check everything and update readmi. I will work on it tomorrow

@Detrous Detrous changed the base branch from master to develop January 13, 2020 21:58
@Detrous
Copy link
Owner

Detrous commented Jan 13, 2020

Could you solve problems with merge conflicts ?

@Vysybyl
Copy link
Contributor Author

Vysybyl commented Jan 14, 2020

Hi!

Conflict solved.
I also dropped the class ClassName(object) syntax since is not needed anymore.

Thanks,

@Detrous
Copy link
Owner

Detrous commented Jan 14, 2020

Thank you. After solve all problem with linters(docstring for all code) i will create new release

@Detrous Detrous merged commit 9cb5c64 into Detrous:develop Jan 14, 2020
Detrous added a commit that referenced this pull request Jan 27, 2020
* Corrected typo in units (#37)

* Added object representation (#39)

* Unused imports (#40)

* isort

* linters ci

* API return objects are now iterable (#38)

* fix README

* Async request fix and other minor changes (#41)

* Corrected typo in units

* Unused imports

* API return objects are now iterable

* Added object representation

* Added darksky__unavailable flag. Not sure about the actual type (bool?)

See https://darksky.net/dev/docs#flags for more details

* Simplified camelCase utility function

* Patched async request yarl type error (List[str])

async make_request now accepts arguments of type List[str]

* Fix requirements & style

* Disabled docstring linter rules

Co-authored-by: Vysybyl <[email protected]>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants