Skip to content

Add flake8-comprehensions to pre-commit #7233

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.

Sign up for GitHub

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

Jump to bottom
Closed
CaedenPH opened this issue Oct 15, 2022 · 1 comment
Closed

Add flake8-comprehensions to pre-commit #7233

CaedenPH opened this issue Oct 15, 2022 · 1 comment
Labels
enhancement This PR modified some existing files

Comments

@CaedenPH
Copy link
Contributor

Feature description

Add flake8-comprehensions to pre-commit and fix errors
@CaedenPH CaedenPH added the enhancement This PR modified some existing files label Oct 15, 2022
CaedenPH added a commit to CaedenPH/Python that referenced this issue Oct 15, 2022
@CaedenPH
ci(pre-commit): Add flake8-comprehensions to pre-commit (TheA…
19edb8d 
…lgorithms#7233)
@CaedenPH CaedenPH mentioned this issue Oct 15, 2022
Merged
14 tasks
CaedenPH added a commit to CaedenPH/Python that referenced this issue Oct 15, 2022
@CaedenPH
fix: Replace map with generator (TheAlgorithms#7233)
77b7d89
cclauss pushed a commit that referenced this issue Oct 15, 2022
@CaedenPH
Add Flake8 comprehensions to pre-commit (#7235)
a652905 
* ci(pre-commit): Add ``flake8-comprehensions`` to ``pre-commit`` (#7233)

* refactor: Fix ``flake8-comprehensions`` errors

* fix: Replace `map` with generator (#7233)

* fix: Cast `range` objects to `list`
@dhruvmanila
Copy link
Member

closed by #7235

Please make sure to link the issue, if existing, in the PR itself, so that it gets closed automatically.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement This PR modified some existing files
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@CaedenPH @dhruvmanila