Skip to content

Replace pylama with prospector #934

Closed

Description

@diraol

Activity

  1. fmv1992 commented on Aug 12, 2018

    @fmv1992
    Contributor

    Sounds very promising.

    It also integrates most of the tools we use so probably we could get rid of the submodules (at least part of them) as well.

  2. fmv1992 commented on Aug 12, 2018

    @fmv1992
    Contributor

    To wrap-it up:

    If we go with prospector:

    1. This issue: how to proceed to clean the hardcoded logilab-common files from this project? #932 gets deprecated.

      On the other hand we would probably have to go with sudo pip3 install ... to make it work and decrease the maintenance effort. I'm very inclined by this idea as updating the submodules everytime is not very elegant, and the plethora of submodules makes the cloning of this repo very lengthy. And once we lose the magic of a single git clone --recurisve command I think we should go with a makefile so that then a single sudo make install does everything.

    2. A lot of the submodules get deprecated.

      • astroid: absorbed by prospector.
      • autopep8: not covered by prospector. Will be kept.
      • mccabe: absorbed by prospector.
      • pycodestyle: not covered by prospector; would probably be abandoned due to high maintenance costs.
      • pydocstyle: absorbed by prospector.
      • pyflakes: absorbed by prospector.
      • pylama: deprecated by the adoption of prospector. Also decreases the burden on the development and maintenance of different modules.
      • pylint: absorbed by prospector.
      • rope: not covered by prospector. Will be kept (not sure).
      • six: not covered by prospector. Will be kept (not sure).
      • snowball_py: deprecated by the adoption of prospector; afaik it is a support tool for other submodules.

      See http://prospector.readthedocs.io/en/latest/supported_tools.html.

    @diraol, am I missing something?

  3. diraol commented on Aug 12, 2018

    @diraol
    ContributorAuthor
    1. I like make install (without sudo)
      I am just not sure about how to proceed to clean the hardcoded logilab-common files from this project? #932, since I don't know what are we using that dependency for.

    2. Submodules:

      • astroid: Agree
      • autopep8: Agree to keep
      • mccabe: Agree
      • pycodestyle: I think that this is covered by `prospector, but aliased to pep8
      • pydocstyle: Agree
      • pyflakes: Agree
      • pylama: Agree \o/
      • pylint: Agree
      • rope: It seems to be a dependency commonly used by our users, it looks like we should keep it.
      • six: Not needed if we drop python2 support.
      • snowball_py: I have no idea for what we use it....
  4. stale commented on Jan 10, 2019

    @stale

    This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

  5. diraol commented on Jan 16, 2019

    @diraol
    ContributorAuthor

    Keeping it up

  6. stale commented on Jun 15, 2019

    @stale

    This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

  7. stale commented on Jul 6, 2019

    @stale

    Closed due to inactivity.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    Type

    No type

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions