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
Feature/pkgdb2 and fas updates #14
Conversation
except AppError: | ||
irc.reply("No such package exists.") | ||
return | ||
branch_list = [] | ||
for listing in pkginfo['packageListings']: | ||
for listing in pkkinfo['packages']: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Typo here: pkkinfo
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ping? :)
Thanks for the ping. Typo fixed. |
@@ -46,7 +46,7 @@ | |||
from fedora.client import ServerError | |||
from fedora.client.fas2 import AccountSystem | |||
from fedora.client.fas2 import FASError | |||
from fedora.client.pkgdb import PackageDB | |||
from pkgdb2client import PkgDB |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Should we explicitly require packagedb-cli somewhere?
one question, otherwise 👍 for me |
Done. There's no setup.py or spec file but I added it to the requirements.txt |
Looks all good to me 😄 👍 |
@abadger should we merge this? |
👍 from me. |
Feature/pkgdb2 and fas updates
FAS and Pkgdb updated last week. This set of changes updates the use of pkgdb to the pkgdb2 api. The FAS update made the query we use slower. This update also increases the client-side timeout that supybot uses so that it can complete the request.