Team-Ampersand / Dotori-iOS

GSM 기숙사 관리 서비스 - 도토리 iOS
https://apps.apple.com/kr/app/id6453122982
MIT License
65 stars 1 forks source link

makeModule(...) -> module(...) 로 바뀌었는데 자동화에 적용이 안되어있어요 #218

Closed HongSJae closed 5 months ago

HongSJae commented 6 months ago

Describe

make module

...

This module has a 'Interface' Target? (y\n, default = n) : y
This module has a 'Testing' Target? (y\n, default = n) : y
This module has a 'UnitTests' Target? (y\n, default = n) : n
This module has a 'UITests' Target? (y\n, default = n) : y
This module has a 'Demo' Target? (y\n, default = n) : n

Bash에서 위 명령어를 사용해 예시로 SignupFeature을 만들 때 아래와 같은 코드가 생성됩니다.

import ProjectDescription
import ProjectDescriptionHelpers
import DependencyPlugin

let project = Project.makeModule(
    name: ModulePaths.Feature.SignupFeature.rawValue,
    product: .staticLibrary,
    targets: [
    .interface(module: .feature(.SignupFeature)),
    .implements(module: .feature(.SignupFeature), dependencies: [
        .feature(target: .SignupFeature, type: .interface)
    ]),
    .testing(module: .feature(.SignupFeature), dependencies: 
        .feature(target: .SignupFeature, type: .interface)
    )
    ],
    internalDependencies: []
)

makeModule(...)module(...)로 수정하고 파라미터 자동화를 수정해야 할 것 같네요..

예시)

import ProjectDescription
import ProjectDescriptionHelpers
import DependencyPlugin

let project = Project.module(
    name: ModulePaths.Feature.SignupFeature.rawValue,
    targets: [
    .interface(module: .feature(.SignupFeature)),
        .implements(module: .feature(.SignupFeature), dependencies: [
        .feature(target: .SignupFeature, type: .interface)
    ]),
    .testing(module: .feature(.SignupFeature), dependencies: [
        .feature(target: .SignupFeature, type: .interface)
    ])
    ]
)

Additional

제가 make Module를 하여 project 파일을 만들면 탭이 비정상적으로 길어지는데..

func tab(_ count: Int) -> String {
    var tabString = ""
    for _ in 0..<count {
        tabString += "\t\t"
    }
    return tabString
}

->

func tab(_ count: Int) -> String {
    var tabString = ""
    for _ in 0..<count {
        tabString += "\t"
    }
    return tabString
}

로 바꾸는 방법은 어떨까요?

HongSJae commented 6 months ago

추가로 Testing 자동화에 괄호가 없어서 수상해요

.testing(module: .feature(.BaseFeature), dependencies: 
            .feature(target: .BaseFeature, type: .interface)
),
uuuunseo commented 5 months ago

피드백 정말 감사합니다!! 현재 해당 문제가 발생한 걸 확인하고 수정 중입니다! 팀원들과 소통 후 빠른 시일내에 반영하겠습니다:)