New webka #37

Merged
Idvon merged 32 commits from issues-36 into main 2023-07-03 10:07:58 +00:00
Member

fix #36

fix #36
Author
Member

Прошу посмотреть пока я фикшу тесты. Кст надо в CI в mypy тест добавить аргумент app.py в строке.

Прошу посмотреть пока я фикшу тесты. Кст надо в CI в mypy тест добавить аргумент app.py в строке.
fix tests
Some checks failed
ci/woodpecker/push/woodpecker Pipeline failed
bcf6b04c73
fix app.py
All checks were successful
ci/woodpecker/push/woodpecker Pipeline was successful
66cc525e05
add mypy app.py tests/
All checks were successful
ci/woodpecker/push/woodpecker Pipeline was successful
66292d711c
d requested changes 2023-06-16 19:17:52 +00:00
d left a comment
Owner

я устал

я устал
@ -35,0 +76,4 @@
config_weather, crd
)
weather_data = net_weather_provider.weather_data( # of the net provider
net_weather_provider.request()
Owner

может сразу вызывать self.request() в weather_data()?

может сразу вызывать self.request() в weather_data()?
Idvon marked this conversation as resolved
@ -44,0 +93,4 @@
geo_config, weather_config, timeout = get_config(file_config)
geo_provider, city_list = get_city_list(geo_config)
print("\n".join([f"{elem}. {city_list[elem]}" for elem in city_list]))
num = int(input("Please write number your city: "))
Owner

"Please write your city number:"
Это ломает возможность запустить погодник в неинтерактивном режиме (по предыдущим тз), когда пользователь просто запускает его из консоли без необходимости что-то вводить

"Please write your city number:" Это ломает возможность запустить погодник в неинтерактивном режиме (по предыдущим тз), когда пользователь просто запускает его из консоли без необходимости что-то вводить
Idvon marked this conversation as resolved
@ -2,2 +3,3 @@
from flask import Flask, render_template
from flask import Flask, redirect, render_template, request, url_for
from wtforms import ( # type: ignore # https://github.com/wtforms/wtforms/issues/618
Owner

А зачем нам втформс если валидация в них не используется и можно обойтись фласковыми формами?

А зачем нам втформс если валидация в них не используется и можно обойтись фласковыми формами?
Idvon marked this conversation as resolved
@ -11,0 +22,4 @@
with open(file_config) as f:
config_data = json.load(f)
config_data["city_name"] = city_name
file_config = Path("net_config.json")
Owner

Это что, это зачем? У нас вроде как все через один конфиг должно настраиваться + это ломает многопользовательский доступ

Это что, это зачем? У нас вроде как все через один конфиг должно настраиваться + это ломает многопользовательский доступ
Idvon marked this conversation as resolved
@ -29,3 +30,1 @@
self.config = dict() if len(data) == 0 else data[0]
else:
self.config = data
self.list_city = get(url, params=payload).json()
Owner

city_list тогда уж

`city_list` тогда уж
Idvon marked this conversation as resolved
@ -30,2 +30,2 @@
else:
self.config = data
self.list_city = get(url, params=payload).json()
if len(self.list_city) == 0:
Owner

if not self.city_list:

if not self.city_list:
Idvon marked this conversation as resolved
@ -32,0 +30,4 @@
self.list_city = get(url, params=payload).json()
if len(self.list_city) == 0:
raise ProviderNoDataError("This city is not found. Please, check city name")
if (isinstance(self.list_city, dict)) and (self.list_city["cod"] is not None):
Owner

а если не isinstance(self.list_city, dict))?

а если не isinstance(self.list_city, dict))?
Author
Member

match get(...):
case []:
raise ...
case {'cod': value, **_}:
raise ...
case valid_list:
self.city_list = valid_list

match get(...): case []: raise ... case {'cod': value, **_}: raise ... case valid_list: self.city_list = valid_list
Idvon marked this conversation as resolved
Owner

Прошу посмотреть пока я фикшу тесты. Кст надо в CI в mypy тест добавить аргумент app.py в строке.

Дык добавь))

> Прошу посмотреть пока я фикшу тесты. Кст надо в CI в mypy тест добавить аргумент app.py в строке. Дык добавь))
d approved these changes 2023-07-03 09:56:46 +00:00
Author
Member

Прошу посмотреть пока я фикшу тесты. Кст надо в CI в mypy тест добавить аргумент app.py в строке.

Дык добавь))

дык добавил

> > Прошу посмотреть пока я фикшу тесты. Кст надо в CI в mypy тест добавить аргумент app.py в строке. > > Дык добавь)) дык добавил
Idvon merged commit 9909d3e946 into main 2023-07-03 10:07:58 +00:00
Idvon deleted branch issues-36 2023-07-03 10:07:59 +00:00
Idvon referenced this pull request from a commit 2023-07-03 10:07:59 +00:00
Sign in to join this conversation.
No description provided.