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

V1.0.0 fixes #167

Merged
merged 6 commits into from
Apr 1, 2021
Merged

V1.0.0 fixes #167

merged 6 commits into from
Apr 1, 2021

Conversation

aafshinfard
Copy link
Member

No description provided.

Now to iterate over a `dict`, the only acceptable way is two call `.items()`
temporarily disabling pylint as it has an active bug for two lines: pylint-dev/pylint#3675
@aafshinfard aafshinfard added the bug Something isn't working label Apr 1, 2021
@aafshinfard aafshinfard self-assigned this Apr 1, 2021
@aafshinfard aafshinfard requested review from jwcodee and lcoombe April 1, 2021 07:52
@aafshinfard
Copy link
Member Author

aafshinfard commented Apr 1, 2021

Three pylint errors due to a pylint active unresolved bug:
pylint-dev/pylint#3675
I fixed this by temporarily disabling the error codes for the lines


+
a warning and
a recommendation at line 2435 (I am not sure I fixed it correctly, please double-check. It suggested that we use generators instead)

@lcoombe
Copy link
Member

lcoombe commented Apr 1, 2021

Looks good to me Amirhossein!
The generator expression fix looks fine to me - I find those really cool and useful in that sort of context!

@aafshinfard aafshinfard merged commit fde0cae into master Apr 1, 2021
@aafshinfard aafshinfard deleted the v1.0.0-fixes branch July 20, 2022 19:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants