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

[symilar] Migrate from getopt to argparse #100

Open
wants to merge 48 commits into
base: main
Choose a base branch
from

Conversation

art049
Copy link
Member

@art049 art049 commented Jun 14, 2024

This pull request was created automatically by CodSpeed to track performance changes of the pull request pylint-dev/pylint#9731.

The original branch is upstream/migrate-from-getopt-to-argparse-in-symilar

Copy link

codspeed-hq bot commented Jun 14, 2024

CodSpeed Performance Report

Merging #100 will not alter performance

Comparing upstream/migrate-from-getopt-to-argparse-in-symilar (44e41d7) with main (dc4b5dd)

Summary

✅ 11 untouched benchmarks

@art049 art049 force-pushed the upstream/migrate-from-getopt-to-argparse-in-symilar branch from 7e17446 to 8e26d68 Compare June 14, 2024 07:30
@art049 art049 force-pushed the upstream/migrate-from-getopt-to-argparse-in-symilar branch from 8e26d68 to 44e41d7 Compare June 17, 2024 07:55
@art049 art049 force-pushed the main branch 4 times, most recently from 3e60a6d to 57fb37f Compare June 21, 2024 07:40
@art049 art049 force-pushed the main branch 4 times, most recently from affb0a3 to 269befd Compare June 28, 2024 07:30
@art049 art049 force-pushed the main branch 4 times, most recently from 6d47585 to 2f97add Compare July 7, 2024 19:45
@art049 art049 force-pushed the main branch 7 times, most recently from d7ea34a to 95f9144 Compare July 11, 2024 19:50
…int-dev#9793)

* Fix a false positive for ``missing-param-doc`` where a method which is decorated with ``typing.overload`` was expected to have a docstring specifying its parameters.

Closes pylint-dev#9739

(cherry picked from commit 9cd5c37)

Co-authored-by: Mark Byrne <[email protected]>
jacobtylerwalls and others added 29 commits July 28, 2024 12:07
updates:
- [github.com/astral-sh/ruff-pre-commit: v0.5.4 → v0.5.5](astral-sh/ruff-pre-commit@v0.5.4...v0.5.5)
Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 4.3.4 to 4.3.5.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](actions/upload-artifact@v4.3.4...v4.3.5)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Updates the requirements on [towncrier](https://github.com/twisted/towncrier) to permit the latest version.
- [Release notes](https://github.com/twisted/towncrier/releases)
- [Changelog](https://github.com/twisted/towncrier/blob/trunk/NEWS.rst)
- [Commits](twisted/towncrier@23.11.0...24.7.1)

---
updated-dependencies:
- dependency-name: towncrier
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
updates:
- [github.com/astral-sh/ruff-pre-commit: v0.5.5 → v0.5.6](astral-sh/ruff-pre-commit@v0.5.5...v0.5.6)
- [github.com/psf/black: 24.4.2 → 24.8.0](psf/black@24.4.2...24.8.0)
- [github.com/pre-commit/mirrors-mypy: v1.11.0 → v1.11.1](pre-commit/mirrors-mypy@v1.11.0...v1.11.1)

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
Fixes:
```
  File "sources/pylint/pylint/checkers/refactoring/refactoring_checker.py", line 2094, in _is_function_def_never_returning
    and node.returns
        ^^^^^^^^^^^^
  File "sources/pylint/.venv/lib/python3.11/site-packages/astroid/bases.py", line 138, in __getattr__
    return getattr(self._proxied, name)
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^
AttributeError: 'Lambda' object has no attribute 'returns'
```

Crash is reproducible if you have something like this:

```python
class C:
    eq = lambda self, y: self == y
```

As a workaround, use a normal function instead of a lambda.

Closes pylint-dev#9865
Bumps [furo](https://github.com/pradyunsg/furo) from 2024.7.18 to 2024.8.6.
- [Release notes](https://github.com/pradyunsg/furo/releases)
- [Changelog](https://github.com/pradyunsg/furo/blob/main/docs/changelog.md)
- [Commits](pradyunsg/furo@2024.07.18...2024.08.06)

---
updated-dependencies:
- dependency-name: furo
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Bumps [actions/upload-artifact](https://github.com/actions/upload-artifact) from 4.3.5 to 4.3.6.
- [Release notes](https://github.com/actions/upload-artifact/releases)
- [Commits](actions/upload-artifact@v4.3.5...v4.3.6)

---
updated-dependencies:
- dependency-name: actions/upload-artifact
  dependency-type: direct:production
  update-type: version-update:semver-patch
...

Signed-off-by: dependabot[bot] <[email protected]>
Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
updates:
- [github.com/astral-sh/ruff-pre-commit: v0.5.6 → v0.5.7](astral-sh/ruff-pre-commit@v0.5.6...v0.5.7)

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
… arguments (pylint-dev#9868)

- add too-many-function-args call
- include handling for too-few-function-args
updates:
- [github.com/astral-sh/ruff-pre-commit: v0.5.7 → v0.6.1](astral-sh/ruff-pre-commit@v0.5.7...v0.6.1)

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
@coco-speed coco-speed force-pushed the upstream/migrate-from-getopt-to-argparse-in-symilar branch from 44e41d7 to 04c34f8 Compare August 24, 2024 11:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants