TouchInstinct
/
LeadKit
302
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

feature/http_status_codes_for_error_responses #5

Merged
ivan.smolin merged 7 commits from feature/http_status_codes_for_error_responses into master 2023-05-24 10:04:08 +03:00
Conversation 0 Commits 7 Files Changed 57 +324 -164
ivan.smolin commented 2023-05-19 18:33:20 +03:00
Member
Copy Link
No description provided.
ivan.smolin added 4 commits 2023-05-19 18:33:22 +03:00
nikita.semenov reviewed 2023-05-19 18:37:16 +03:00
Plugins/TISwiftLintPlugin/plugin.swift Outdated
@ -0,0 +42,4 @@
"SCRIPT_INPUT_FILE_COUNT": "1",
"SCRIPT_INPUT_FILE_0": target.directory.removingLastComponent().lastComponent,
// "FORCE_LINT": "1", // Lint all files in target (not only modified)
"AUTOCORRECT": "1" //
nikita.semenov commented 2023-05-19 18:37:16 +03:00
Member
Copy Link

оставшиеся комменты - ок?

оставшиеся комменты - ок?
ivan.smolin commented 2023-05-19 18:43:18 +03:00
Author
Member
Copy Link

Да, предполагается, что тут как и в проектах будет инкрементальный линтинг, но если хочеться поправить всё - то просто расскоментировать.
Ну последний пустой коммент это артефакт конечно, но пофикшу если будут еще комменты

Да, предполагается, что тут как и в проектах будет инкрементальный линтинг, но если хочеться поправить всё - то просто расскоментировать. Ну последний пустой коммент это артефакт конечно, но пофикшу если будут еще комменты
nikita.semenov marked this conversation as resolved
nikita.semenov reviewed 2023-05-19 18:51:22 +03:00
TINetworking/Sources/Mapping/BodySerializer/ApplicationJsonBodySerializer.swift
@ -42,3 +42,3 @@
let mimeType = CommonMediaTypes.applicationJson.rawValue
guard let body = body else {
guard let body else {
nikita.semenov commented 2023-05-19 18:51:14 +03:00
Member
Copy Link

Как так получается, что в подфайлах у нас версия swift 5.3, а конструкции используем из 5.7?

Как так получается, что в подфайлах у нас версия swift 5.3, а конструкции используем из 5.7?
👍 1 👀 1
nikita.semenov marked this conversation as resolved
vladimir.makarov reviewed 2023-05-22 10:27:56 +03:00
TINetworking/Sources/Request/SerializedRequest.swift
@ -67,3 +63,1 @@
lhs.headers == rhs.headers &&
lhs.cookies == rhs.cookies &&
lhs.acceptableStatusCodes == rhs.acceptableStatusCodes
lhs.path == rhs.path &&
vladimir.makarov commented 2023-05-22 10:27:56 +03:00
Member
Copy Link

Не очень красиво он здесь форматирует, конечно 🙂

Не очень красиво он здесь форматирует, конечно 🙂
ivan.smolin commented 2023-05-22 13:04:05 +03:00
Author
Member
Copy Link

что поделать 🤷

что поделать 🤷
vladimir.makarov marked this conversation as resolved
vladimir.makarov reviewed 2023-05-22 10:30:58 +03:00
ivan.smolin added 1 commit 2023-05-22 11:57:31 +03:00
vladimir.makarov approved these changes 2023-05-22 13:00:11 +03:00
ivan.smolin added 1 commit 2023-05-22 13:35:08 +03:00
nikita.semenov approved these changes 2023-05-23 09:03:49 +03:00
ivan.smolin added 1 commit 2023-05-24 09:28:25 +03:00
vladimir.makarov approved these changes 2023-05-24 10:03:48 +03:00
ivan.smolin merged commit 0e0a8d733e into master 2023-05-24 10:04:08 +03:00
ivan.smolin deleted branch feature/http_status_codes_for_error_responses 2023-05-24 10:04:09 +03:00
Sign in to join this conversation.
Reviewers
No reviewers
nikita.semenov
vladimir.makarov
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: TouchInstinct/LeadKit#5
No description provided.
Delete Branch "feature/http_status_codes_for_error_responses"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?