Tadaboody/good_smell issues and pull requests

#75 - Feature: Calling good_smell with multiple files

Issue - State: open - Opened by qequ about 2 years ago
Labels: enhancement, new

#74 - Add smell: Warn when `str.join`ing a constant amount of members and prefer to str.format.

Pull Request - State: closed - Opened by Tadaboody over 3 years ago
Labels: smell

#73 - :bug: Fix bad subscript attribute in range len fix

Pull Request - State: closed - Opened by Tadaboody over 3 years ago

#72 - Fix contributing guide to use poetry

Pull Request - State: closed - Opened by Tadaboody over 4 years ago - 2 comments

#71 - Point "contributing guide" link to correct file

Pull Request - State: closed - Opened by ghost over 4 years ago

#70 - Add test for unpacking in for loop.

Pull Request - State: closed - Opened by Dreamsorcerer over 4 years ago - 5 comments

#69 - :bug: Export entry points again

Pull Request - State: closed - Opened by Tadaboody over 4 years ago

#68 - Move to github actions

Pull Request - State: closed - Opened by Tadaboody over 4 years ago - 1 comment

#67 - Bookkeeping

Pull Request - State: closed - Opened by Tadaboody over 4 years ago - 1 comment

#66 - Don't transform ast in linting plugin

Pull Request - State: closed - Opened by Tadaboody over 4 years ago - 1 comment

#65 - Frozen requirements.txt breaks other plugins

Issue - State: closed - Opened by Dreamsorcerer over 4 years ago - 6 comments
Labels: bug, new

#64 - Breaks other flake8 plugins by manipulating the AST.

Issue - State: closed - Opened by Dreamsorcerer over 4 years ago
Labels: bug, new

#63 - Docs link broken

Issue - State: closed - Opened by Dreamsorcerer over 4 years ago - 1 comment
Labels: bug, new

#62 - Crash under very specific scenario

Issue - State: open - Opened by An-dz about 5 years ago
Labels: bug, new

#61 - Tuple object has no attribute id

Issue - State: closed - Opened by Dreamsorcerer over 5 years ago - 1 comment
Labels: bug, new

#60 - Simplify collection with case end symbol

Pull Request - State: closed - Opened by Tadaboody almost 6 years ago - 1 comment

#59 - Convert AstSmell to use instancing instead of inheritance

Issue - State: closed - Opened by Tadaboody almost 6 years ago
Labels: refactoring

#58 - YieldFrom Smell

Pull Request - State: closed - Opened by AdamGold almost 6 years ago - 6 comments

#57 - Missing dropwhile

Pull Request - State: closed - Opened by Tadaboody almost 6 years ago - 2 comments

#56 - Use dropwhile

Issue - State: closed - Opened by Tadaboody almost 6 years ago - 1 comment
Labels: smell, new

#55 - Replace for-append with generator

Issue - State: open - Opened by Tadaboody almost 6 years ago
Labels: smell, new

#54 - Use argparse instead of fire for a cleaner interface and less dependancies

Issue - State: open - Opened by Tadaboody almost 6 years ago
Labels: enhancement, new

#53 - Use with open instead of manually opening and closing file handler

Issue - State: open - Opened by Tadaboody almost 6 years ago
Labels: smell, new

#52 - Use builtin breakpoint()

Issue - State: open - Opened by Tadaboody almost 6 years ago
Labels: smell

#51 - Simplify with yield from

Issue - State: closed - Opened by Tadaboody almost 6 years ago - 2 comments
Labels: good first issue, smell

#50 - Support multiple python versions

Issue - State: open - Opened by Tadaboody almost 6 years ago - 1 comment
Labels: enhancement, new

#49 - Fix smells in-place

Issue - State: open - Opened by Tadaboody almost 6 years ago
Labels: enhancement, new

#48 - Have the fixer work with other auto-fomratters

Issue - State: open - Opened by Tadaboody almost 6 years ago
Labels: enhancement, new

#47 - Add a pre-commit hook config

Issue - State: open - Opened by Tadaboody almost 6 years ago

#46 - Don't flatten a nested for with dependencies

Pull Request - State: closed - Opened by Tadaboody almost 6 years ago - 1 comment

#45 - Add filter to existing gen exp

Pull Request - State: closed - Opened by Tadaboody almost 6 years ago - 1 comment

#44 - adding to an existing generator exp (important for chaining with #7 )

Issue - State: closed - Opened by Tadaboody almost 6 years ago

#43 - Cov setup

Pull Request - State: closed - Opened by Tadaboody almost 6 years ago - 1 comment

#42 - Create a script to auto-generate the smell list

Pull Request - State: closed - Opened by Tadaboody almost 6 years ago

#41 - Add more cases of range-len

Pull Request - State: closed - Opened by Tadaboody almost 6 years ago

#40 - Filter

Pull Request - State: closed - Opened by Tadaboody almost 6 years ago - 1 comment

#39 - Get rid of clunky generator in test collection

Issue - State: closed - Opened by Tadaboody almost 6 years ago
Labels: refactoring

#38 - Refactoring

Pull Request - State: closed - Opened by Tadaboody almost 6 years ago

#37 - Add black pre-commit hook

Issue - State: closed - Opened by Tadaboody almost 6 years ago

#36 - Test collection

Pull Request - State: closed - Opened by Tadaboody almost 6 years ago

#35 - Use setuptools.find_packages to include all python packages

Pull Request - State: closed - Opened by Tadaboody almost 6 years ago

#34 - `smells` package not included in dist

Issue - State: closed - Opened by Tadaboody almost 6 years ago
Labels: bug

#33 - Add requirements.txt

Pull Request - State: closed - Opened by Tadaboody almost 6 years ago - 1 comment

#32 - Create a CHANGELOG

Issue - State: open - Opened by Tadaboody almost 6 years ago

#31 - Use stlib zfill to lead 0s

Pull Request - State: closed - Opened by Tadaboody almost 6 years ago

#30 - Move smells to a subdir

Pull Request - State: closed - Opened by Tadaboody almost 6 years ago - 1 comment

#29 - Creating the error code calls log10(0)

Issue - State: closed - Opened by Tadaboody almost 6 years ago - 1 comment

#28 - Move smells to sub-directory

Issue - State: closed - Opened by MeitarR almost 6 years ago

#27 - Add requirements.txt

Issue - State: closed - Opened by MeitarR almost 6 years ago

#26 - Don't flatten a nested for with dependencies

Issue - State: closed - Opened by Tadaboody almost 6 years ago
Labels: bug

#25 - Flake8 plugin

Pull Request - State: closed - Opened by Tadaboody almost 6 years ago

#24 - Create plugins for existing linters

Issue - State: closed - Opened by Tadaboody almost 6 years ago

#23 - Properly test warnings

Issue - State: closed - Opened by Tadaboody almost 6 years ago - 1 comment

#22 - Warn about smells - #17

Pull Request - State: closed - Opened by Tadaboody almost 6 years ago - 1 comment

#21 - Add option to revert changes

Issue - State: open - Opened by Tadaboody almost 6 years ago

#20 - Use f-string instead of string.format

Issue - State: open - Opened by Tadaboody almost 6 years ago - 1 comment
Labels: smell

#19 - Pypi release

Pull Request - State: closed - Opened by Tadaboody almost 6 years ago

#18 - Create project website

Issue - State: open - Opened by Tadaboody almost 6 years ago

#17 - Properly warn about smells like a linting tool

Issue - State: closed - Opened by Tadaboody almost 6 years ago - 3 comments
Labels: enhancement

#16 - Rename internally to GoodSmell

Pull Request - State: closed - Opened by Tadaboody almost 6 years ago

#15 - Add CONTRIBUTING guide

Issue - State: closed - Opened by Tadaboody almost 6 years ago

#14 - Nested for to prod 12

Pull Request - State: closed - Opened by Tadaboody almost 6 years ago

#13 - Support replacing range arguments with enumerate

Issue - State: open - Opened by Tadaboody almost 6 years ago

#12 - Replace nested for with product

Issue - State: closed - Opened by Tadaboody almost 6 years ago - 1 comment
Labels: smell

#11 - (WIP) Add a fixer for range-len

Pull Request - State: closed - Opened by OmerBelsky almost 6 years ago - 1 comment

#10 - Configure travis test runner

Pull Request - State: closed - Opened by Tadaboody almost 6 years ago

#9 - Make mypy-compliant typing

Issue - State: closed - Opened by Tadaboody almost 6 years ago - 1 comment
Labels: enhancement

#8 - Missing takewhile

Issue - State: open - Opened by Tadaboody almost 6 years ago
Labels: smell

#7 - Move map to Iterator

Issue - State: open - Opened by Tadaboody almost 6 years ago - 1 comment
Labels: help wanted, smell

#6 - Missing filter

Issue - State: closed - Opened by Tadaboody almost 6 years ago
Labels: good first issue, smell

#5 - Add fix for range-len-assign smell

Pull Request - State: closed - Opened by Tadaboody almost 6 years ago

#4 - Fix range_len_assign smell

Issue - State: closed - Opened by Tadaboody almost 6 years ago
Labels: smell

#3 - Imploy a Ci/Cd pipeline

Issue - State: closed - Opened by Tadaboody almost 6 years ago

#2 - Run tests on open source samples

Issue - State: open - Opened by Tadaboody almost 6 years ago
Labels: help wanted

#1 - Support comment flags to disable features

Issue - State: closed - Opened by Tadaboody almost 6 years ago - 1 comment
Labels: enhancement

GitHub / Tadaboody/good_smell issues and pull requests