search

fedora-infra / fedora-messaging

A library for sending AMQP messages with JSON schema in Fedora infrastructure
GNU General Public License v2.0
53 stars 52 forks source link

Add missing ‹requests› dependency to pyproject.toml #365

Closed mfocko closed 3 months ago

mfocko commented 4 months ago

Related to #364 Related to packit/agile#552

jeremycline commented 4 months ago

Ah, you'll need to check in the poetry.lock file as well.

mfocko commented 3 months ago

Ah, you'll need to check in the poetry.lock file as well.

I'll have a look at that, it got lost among the other notifications

mfocko commented 3 months ago

Not sure where it's coming from, but I hit this:

fedora-messaging % poetry lock
Updating dependencies
Resolving dependencies... (4.8s)

Package docutils (0.21.post1) not found.
jeremycline commented 3 months ago

I'm not particularly familiar with poetry, but the CI logs said try poetry lock --no-update which seems to satisfy the tests locally.

mfocko commented 3 months ago

Ah, OK… it works with --no-update :facepalm: it was in brackets, so I assumed it's optional