Skip to content

Commit 369a052

Browse files
author
Sourcery AI
committed
'Refactored by Sourcery'
1 parent c6f2fb2 commit 369a052

File tree

49 files changed

+140
-223
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

49 files changed

+140
-223
lines changed

days/009-012-modern-apis-starred/demo/app.py

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -14,13 +14,9 @@ def _load_cars_data():
1414

1515

1616
cars = _load_cars_data()
17-
VALID_MANUFACTURERS = set([car["manufacturer"]
18-
for car in cars.values()])
17+
VALID_MANUFACTURERS = {car["manufacturer"] for car in cars.values()}
1918
CAR_NOT_FOUND = 'Car not found'
2019

21-
# definition
22-
23-
2420
class Car(types.Type):
2521
id = validators.Integer(allow_null=True) # assign in POST
2622
manufacturer = validators.String(enum=list(VALID_MANUFACTURERS))

days/009-012-modern-apis-starred/demo/test_app.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -52,7 +52,7 @@ def test_create_car():
5252
def test_create_car_after_delete():
5353
"""Test to fail create_car's len(cars)+1 (fix max(cars.keys())+1)"""
5454
car_count = len(cars)
55-
response = client.delete(f'/99/')
55+
response = client.delete('/99/')
5656
assert response.status_code == 204
5757
assert len(cars) == car_count - 1
5858
data = {'manufacturer': 'Honda',

days/017-020-flask-call-apis/code/program/routes.py

Lines changed: 3 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55

66
from program import app
77

8-
time_now = str(datetime.today())
8+
time_now = str(datetime.now())
99

1010

1111
@app.route('/')
@@ -43,11 +43,6 @@ def get_chuck_joke():
4343

4444

4545
def get_poke_colours(colour):
46-
r = requests.get('https://pokeapi.co/api/v2/pokemon-color/' + colour.lower())
46+
r = requests.get(f'https://pokeapi.co/api/v2/pokemon-color/{colour.lower()}')
4747
pokedata = r.json()
48-
pokemon = []
49-
50-
for i in pokedata['pokemon_species']:
51-
pokemon.append(i['name'])
52-
53-
return pokemon
48+
return [i['name'] for i in pokedata['pokemon_species']]

days/021-024-quart-async/async_cityscape_api/app.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@ def configure_app():
2121
services.sun_service.use_cached_data = data.get('use_cached_data', False)
2222
services.location_service.use_cached_data = True
2323

24-
print("Using cached data? {}".format(data.get('use_cached_data', False)))
24+
print(f"Using cached data? {data.get('use_cached_data', False)}")
2525

2626

2727
def run_web_app():

days/021-024-quart-async/async_cityscape_api/services/location_service.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,5 +30,5 @@ async def get_lat_long(zip_code: str, country: str) -> Tuple[float, float]:
3030
resp.raise_for_status()
3131
data = await resp.json()
3232

33-
city_data = data.get(f'{zip_code}, {country}', dict())
33+
city_data = data.get(f'{zip_code}, {country}', {})
3434
return city_data.get('latitude', 0.00), city_data.get('longitude', 0.00)

days/021-024-quart-async/cityscape_api/app.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ def configure_app():
2323
# Sadly, datasciencetoolkit.org seems to have gone out of existence.
2424
services.location_service.use_cached_data = True
2525

26-
print("Using cached data? {}".format(data.get('use_cached_data', False)))
26+
print(f"Using cached data? {data.get('use_cached_data', False)}")
2727

2828

2929
def run_web_app():

days/021-024-quart-async/cityscape_api/services/location_service.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,5 +23,5 @@ def get_lat_long(zip_code: str, country: str) -> Tuple[float, float]:
2323

2424
data = resp.json()
2525

26-
city_data = data.get(f'{zip_code}, {country}', dict())
26+
city_data = data.get(f'{zip_code}, {country}', {})
2727
return city_data.get('latitude', 0.00), city_data.get('longitude', 0.00)

days/021-024-quart-async/python_async/async_program.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66

77
def main():
88
t0 = datetime.datetime.now()
9-
print(colorama.Fore.WHITE + "App started.", flush=True)
9+
print(f"{colorama.Fore.WHITE}App started.", flush=True)
1010

1111
# Changed from asyncio.get_event_loop() => asyncio.new_event_loop()
1212
# There has been a deprecation:
@@ -31,7 +31,7 @@ async def generate_data(num: int, data: asyncio.Queue):
3131
item = idx*idx
3232
await data.put((item, datetime.datetime.now()))
3333

34-
print(colorama.Fore.YELLOW + f" -- generated item {idx}", flush=True)
34+
print(f"{colorama.Fore.YELLOW} -- generated item {idx}", flush=True)
3535
await asyncio.sleep(random.random() + .5)
3636

3737

days/021-024-quart-async/python_async/sync_program.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,7 @@
66

77
def main():
88
t0 = datetime.datetime.now()
9-
print(colorama.Fore.WHITE + "App started.", flush=True)
9+
print(f"{colorama.Fore.WHITE}App started.", flush=True)
1010
data = []
1111
generate_data(20, data)
1212
process_data(20, data)
@@ -20,7 +20,7 @@ def generate_data(num: int, data: list):
2020
item = idx*idx
2121
data.append((item, datetime.datetime.now()))
2222

23-
print(colorama.Fore.YELLOW + f" -- generated item {idx}", flush=True)
23+
print(f"{colorama.Fore.YELLOW} -- generated item {idx}", flush=True)
2424
time.sleep(random.random() + .5)
2525

2626

days/021-024-quart-async/your_turn/day_2/web_crawl/program.py

Lines changed: 4 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44

55

66
def get_html(episode_number: int) -> str:
7-
print(Fore.YELLOW + f"Getting HTML for episode {episode_number}", flush=True)
7+
print(f"{Fore.YELLOW}Getting HTML for episode {episode_number}", flush=True)
88

99
url = f'https://talkpython.fm/{episode_number}'
1010
resp = requests.get(url)
@@ -14,13 +14,10 @@ def get_html(episode_number: int) -> str:
1414

1515

1616
def get_title(html: str, episode_number: int) -> str:
17-
print(Fore.CYAN + f"Getting TITLE for episode {episode_number}", flush=True)
17+
print(f"{Fore.CYAN}Getting TITLE for episode {episode_number}", flush=True)
1818
soup = bs4.BeautifulSoup(html, 'html.parser')
1919
header = soup.select_one('h1')
20-
if not header:
21-
return "MISSING"
22-
23-
return header.text.strip()
20+
return "MISSING" if not header else header.text.strip()
2421

2522

2623
def main():
@@ -33,7 +30,7 @@ def get_title_range():
3330
for n in range(150, 170):
3431
html = get_html(n)
3532
title = get_title(html, n)
36-
print(Fore.WHITE + f"Title found: {title}", flush=True)
33+
print(f"{Fore.WHITE}Title found: {title}", flush=True)
3734

3835

3936
if __name__ == '__main__':

0 commit comments

Comments
 (0)