Prospector

-------------

Mea culpa release

**Fix**

The effort to allow pylint configuration in ``pyproject.toml`` to be used as an external config source (`issue here <https://github.com/PyCQA/prospector/issues/485>`_) had the unintended side effect where any project using poetry would now use that configuration and thus would ignore the pylint configuration in the profile. This was true even if the ``pyproject.toml`` had no pylint directives in it.

The behaviour has now been fixed where pylint will be configured using configuration from the profile *first* and then if any additional settings are found in a ``pylintrc`` or ``pyproject.toml`` or ``setup.cfg`` then these will override the profile configuration, instead of replacing it entirely.

This also has the benefit of fixing `227 <https://github.com/PyCQA/prospector/issues/227>`_.

-------------

The war on bugs.

**Fixes**:

* Autodetect now does not die if a user does not have permissions (related to `271 <https://github.com/PyCQA/prospector/issues/271>`_ and `#487 <https://github.com/PyCQA/prospector/issues/487>`_)
* Fixed that some pylint documentation warning messages were not correctly included in the list of documentation warnings to squash if doc warnings are not desired.
* Fixed the exit code for prospector - it was always ``0`` after the move to using poetry for packaging instead of ``1`` if errors were found (unless ``--zero-exit``) was used. This now exits with the correct code based on the documented (and previous) behaviour.
* Fix that ``pep8`` would overwrite instead of inherit from previous ``pycodestyle`` blocks, same with pep257 - `491 (comment) <https://github.com/PyCQA/prospector/issues/491#issuecomment-1053539711>`_
* Fix the pre-commit hook, as it could not run without being installed ``[with_everything]``, due to the "NotAvailableTool" class not properly implementing the abstract base class.
* Improved documentation about the pre-commit hook as well to clarify its use better - `484 <https://github.com/PyCQA/prospector/issues/484>`_

-------------

More bugfixes!

**Fixes**:

* Fix that ``pep8`` and ``pep257`` sections were renamed but the old deprecated values were not properly used to configure ``pycodestyle`` and ``pydocstyle`` - `491 <https://github.com/PyCQA/prospector/issues/491>`_
* Better handling for when the user running prospector is not able to read a file or directory - `271 <https://github.com/PyCQA/prospector/issues/271>`_ and `#487 <https://github.com/PyCQA/prospector/issues/487>`_

-------------

Lots of smaller bugfixes.

**Fixes**:

* Prospector now configures pylint using settings found in ``pyproject.toml`` or ``setup.cfg``, not only ``.pylintrc`` - `485 <https://github.com/PyCQA/prospector/issues/485>`_
* Fixed ``--no-style-warnings`` command line argument no longer warning after renaming ``pep8`` to ``pycodestyle`` - `488 <https://github.com/PyCQA/prospector/issues/488>`_
* Documentation is building again - `473 <https://github.com/PyCQA/prospector/issues/473>`_
* ``--with-tool`` flag now respects - but overrides - tools disabled in profiles - `447 <https://github.com/PyCQA/prospector/issues/447>`_
* Fixed crash with merging multiple import warnings - `477 <https://github.com/PyCQA/prospector/issues/477>`_
* Fixed segfault when analysing code using cartopy - `403 <https://github.com/PyCQA/prospector/issues/403>`_

-------------

This is mostly a "tidying up" release but some things have changed which may cause differences to output, hence the bump of the major version.

**New**:

* Added a ``--quiet`` command line option to suppress all output. Useful if you just want to know the exit code of prospector for scripting.
* Removed the prospector "indent checker" since this is now no longer in pylint `482 <https://github.com/PyCQA/prospector/issues/482>`_

**Fixes**:

`Deprecation warning:`

* Tools ``pep8`` and ``pep257`` have been renamed to ``pycodestyle`` and ``pydocstyle`` respectively. This is because the tools themselves were renamed years ago - See `222 <https://github.com/PyCQA/prospector/issues/222>`_.

Note that this means that prospector profiles and message output uses this new name instead of the old name, so you will need to update your configuration. The old names will still work, but this legacy behaviour will be removed in prospector 2.0

* There is now a ``--legacy-tool-names`` flag for outputting pep8 or pep257 as the tool name when outputting errors. This is to be backwards compatible with any parsing logic; this flag is also deprecated and will go away in prospector 2.0

**Tidying up internals**

These are all internal prospector code quality improvements. Ideally, they should not be noticed by anybody as they are internal refactorings.

* `467 <https://github.com/PyCQA/prospector/issues/467>`_ - Removed nosetests, as nose is not compatible with Python 3.10 yet and the pytest tests were already doing the same thing
* Tidied up the tox testing
* Started adding some type hints to methods
* Fixed lots of warnings raised by prospector when running prospector on itself...
* Removed some old python2 compatibility code which is no longer needed now python2 is not supported at all
* Fixed hyperlink formatting in this CHANGELOG to be RST (was never updated after converting from markdown)
* Replaced `os.path` with `pathlib.Path` everywhere in prospector internals, to improve and simplify finding files to inspect. Theoretically this behaves in the same way as far as the user will see (please open a ticket if you notice anything obviously different)

-------------

- Update pyflakes to 2.* `454 <https://github.com/PyCQA/prospector/issues/454)>`_