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

feed.news(3) argument bug #32

Open
InnocentAnyaele opened this issue Apr 25, 2023 · 0 comments
Open

feed.news(3) argument bug #32

InnocentAnyaele opened this issue Apr 25, 2023 · 0 comments
Labels
bug Something isn't working

Comments

@InnocentAnyaele
Copy link

Bug description
When using the package in the cli to display news feed and you pass in a news number count like "pprint.pprint(f.news(3))" just as it says in the read.me, It shows an error "feed.news() takes 1 positional argument but 2 were given". I went through the code and noticed the news argument in the feed class (found in defe/defe.py) doesn't take in an argument for the number of news, that's why this error is showing, so I'm suggesting you update the package usage in the read.me to reflect that it doesn't take in any argument.

Steps to reproduce the behavior:
from defe import defe
import pprint
f = defe.feed()
pprint.pprint(f.news(3))
You should see a positional argument error.

Expected behavior
Update read.me to reflect that no positional argument is passed.

image

Desktop (please complete the following information):

  • OS: [Windows]
  • Terminal

Additional context
I may not have passed the argument right, but following the documentation from the read.me, this is what it reproduces.

@InnocentAnyaele InnocentAnyaele added the bug Something isn't working label Apr 25, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

1 participant