Jiri Popelka

test_httpretty_should_handle_paths_starting_with_two_slashes needs update

2

[The test checks that we can handle URLs with paths starting with //](https://github.com/gabrielfalcao/HTTPretty/blob/7804c533ec344eeefaacef5340fe479858f7154b/tests/functional/test_requests.py#L949). However the [http.server.BaseHTTPRequestHandler.parse_request() now replaces '//' with '/'](https://github.com/python/cpython/pull/93879) so the test should be updated to allow also...

Add https://github.com/packit/ogr to 'Projects using PyGithub'

1

BTW, the http://devassistant.org says that 'DevAssistant is dead' and [its repo](https://github.com/devassistant/devassistant) has been archived (the last commit is 6 years old) so you might consider removing it from the list...

[source-git update-source-git] can't sync a specfile change due to different context

3

Yesterday, when I run `source-git update-source-git` to sync some dist-git changes one of the patches (created from a dist-git commit) failed to apply. The reason was that in the dist-git...

Run mypy without --no-strict-optional

7

1. remove [`--no-strict-optional` from pre-commit-config](https://github.com/packit/packit/blob/623077c691d7f3588fae6767e9eb8f77dcaec085/.pre-commit-config.yaml#L50) 2. run `pre-commit run --all-files` 3. fix all errors https://mypy.readthedocs.io/en/stable/kinds_of_types.html#no-strict-optional See also packit/ogr#696 & packit/packit-service#1433

From: [user-cont-conu-pr/1352](https://ci.centos.org/job/user-cont-conu-pr/1352/consoleFull) * `TypeError: delete_namespaced_pod() takes 3 positional arguments but 4 were given` * `TypeError: delete_namespace() takes 2 positional arguments but 3 were given`

intermittent test_cleanup_containers() failing

2

For example [here](https://ci.centos.org/job/user-cont-conu-master/15/consoleFull): ``` 11:47:45 =================================== FAILURES =================================== 11:47:45 ___________________________ test_cleanup_containers ____________________________ 11:47:45 11:47:45 def test_cleanup_containers(): 11:47:45 with DockerBackend(logging_level=logging.DEBUG, cleanup=[CleanupPolicy.CONTAINERS]) as backend: 11:47:45 # cleaning up from previous runs 11:47:45...

Insufficient intructions in tests_recording/README.md

4

[It says](https://github.com/packit/packit/tree/main/tests_recording#command-to-run): "Ensure to remove files and directories for tests what you would like to regenerate." Ok, but there are yaml files, tar.xz files and symlinks. Which of those should...

Use dashes instead of asterisks when generating changelog entries

1

[Prettier](https://github.com/prettier/prettier) does not like the asterisks.

Bump also spec file version

1

When creating a new release PR, it'd be nice if release-bot bumped also a version in a spec file if there's one in the repo. Like [here](https://github.com/packit-service/ogr/pull/78/files). `rpmdev-bumpspec` (from `rpmdevtools`...

Since #237 release-bot will be doing the hard work in Celery tasks/workers. The [Configuration.set_logging()](https://github.com/user-cont/release-bot/blob/master/release_bot/configuration.py#L61) should be improved/changed as well so that it works nicely in the Celery task/worker. See: *...