-
Notifications
You must be signed in to change notification settings - Fork 10
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
SPT-1998 Swift cuncurrency support #122
Conversation
b9f67bc
to
e1f2924
Compare
15513ec
to
85c91ed
Compare
e1f2924
to
d6d531f
Compare
85c91ed
to
45008c3
Compare
d6d531f
to
6a5629e
Compare
45008c3
to
74883a8
Compare
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.
Несколько замечаний по тестам (в основном некритичные) и много опечаток в доке (возможно не все нашел)
NodeKitTests/UnitTests/MockerIntegration/MockerProxyConfigNodeTests.swift
Show resolved
Hide resolved
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.
тесты не глядел, потому уже с другими тестами гляну
не помню, было ли что-то критичное в комментах, кажется как будто если где-то и были неточности - ты тестами потом найдешь) так что если ничего критичного и работа будет блочиться - можешь мержить
request: urlRequest, | ||
response: urlResponse, | ||
data: value, | ||
metrics: nil, // ?? почему nil |
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.
тоже интересно стало))
cf47973
to
d3ae0e1
Compare
Добавлена поддержка Swift Concurrency.
Созданы новые протоколы:
Поправлены тесты для прохождения ci. Основные работы по тестам будут в следующих ПР.