Skip to content

Fix/fixes get top billionaries code #11466

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.

Sign up for GitHub

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

Merged
merged 9 commits into from
Jun 16, 2024
Merged

Fix/fixes get top billionaries code #11466

Changes from 6 commits
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
cf38983
fix: modify the depracated code and add new tests
ra230537 Jun 16, 2024
a9a5786
fix: remove test from pr
ra230537 Jun 16, 2024
ebb5f02
fix: remove the useless utc import
ra230537 Jun 16, 2024
d20730e
fix: add explicit tz argument
ra230537 Jun 16, 2024
9809d95
fix: fixes ruff checking
ra230537 Jun 16, 2024
fa4d963
Remove UP017 #noqa comments from code
ra230537 Jun 16, 2024
5c76cb4
Update get_top_billionaires.py
cclauss Jun 16, 2024
9198f4f
[pre-commit.ci] auto fixes from pre-commit.com hooks
pre-commit-ci[bot] Jun 16, 2024
1897916
Update get_top_billionaires.py
cclauss Jun 16, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 6 additions & 2 deletions web_programming/get_top_billionaires.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,14 +3,17 @@
This works for some of us but fails for others.
"""

from datetime import UTC, date, datetime
import doctest
from datetime import date, datetime, timedelta, timezone

import requests
from rich import box
from rich import console as rich_console
from rich import table as rich_table

LIMIT = 10
UTC_OFFSET = timedelta(hours=0)
UTC = timezone(UTC_OFFSET)
TODAY = datetime.now(tz=UTC)
API_URL = (
"https://www.forbes.com/forbesapi/person/rtb/0/position/true.json"
Expand Down Expand Up @@ -65,7 +68,7 @@ def get_forbes_real_time_billionaires() -> list[dict[str, int | str]]:
"Country": person["countryOfCitizenship"],
"Gender": person["gender"],
"Worth ($)": f"{person['finalWorth'] / 1000:.1f} Billion",
"Age": years_old(person["birthDate"]),
"Age": str(years_old(person["birthDate"] / 1000)),
}
for person in response_json["personList"]["personsLists"]
]
Expand Down Expand Up @@ -95,4 +98,5 @@ def display_billionaires(forbes_billionaires: list[dict[str, int | str]]) -> Non


if __name__ == "__main__":
doctest.testmod()
display_billionaires(get_forbes_real_time_billionaires())