-
Notifications
You must be signed in to change notification settings - Fork 0
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
[TNT-80] Static Library에서도 Preview를 사용 가능하도록 Tuist 수정 #29
Changes from 2 commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 안 쓰이는 내용은 정리해주시면 좋을 것 같아요! There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 넵! |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,14 @@ | ||
// | ||
// Tuist.swift | ||
// Manifests | ||
// | ||
// Created by 박서연 on 1/22/25. | ||
// | ||
|
||
import ProjectDescription | ||
|
||
let tuist = Tuist.init( | ||
project: .tuist( | ||
compatibleXcodeVersions: ["16.2"] | ||
) | ||
) | ||
Comment on lines
+10
to
+14
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 이런 세팅이 있는 줄은 처음 알았네요..! 협업 시 너무 좋은 것 같습니다 👍👍 |
This file was deleted.
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -8,44 +8,38 @@ | |
@preconcurrency import ProjectDescription | ||
|
||
let dependencyInfo: [DependencyInformation: [DependencyInformation]] = [ | ||
.TnTApp: [.Presentation], | ||
.TnTApp: [.Presentation, .DesignSystem], | ||
.Presentation: [.DesignSystem, .Domain, .ComposableArchitecture], | ||
.Domain: [.Data, .SwiftDepedencies, .KakaoSDKUser], | ||
.Data: [], | ||
.DesignSystem: [.ComposableArchitecture, .Lottie], | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 요부분 관련해서 전에 얘기했었던 내용들도 추가로 반영해주실 수 있을까요?!
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 이것도.. 이것저것 테스트하다가 추가된 것같은데 말씀해주신 사항 수정해놓을게요. |
||
.DI: [] | ||
] | ||
|
||
public enum DependencyInformation: String, CaseIterable, Sendable { | ||
case TnTApp = "TnTApp" | ||
case Presentation = "Presentation" | ||
case Domain = "Domain" | ||
case Data = "Data" | ||
case DI = "DI" | ||
case DesignSystem = "DesignSystem" | ||
case Lottie = "Lottie" | ||
case ComposableArchitecture = "ComposableArchitecture" | ||
case KakaoSDKUser = "KakaoSDKUser" | ||
case SwiftDepedencies = "Dependencies" | ||
} | ||
|
||
extension DependencyInformation { | ||
public func setDependency(module: DependencyInformation) -> TargetDependency { | ||
switch self { | ||
case .TnTApp: | ||
return .project(target: self.rawValue, path: "Projects/TnTApp") | ||
case .Presentation: | ||
return .project(target: self.rawValue, path: "Projects/Presentation") | ||
case .Domain: | ||
return .project(target: self.rawValue, path: "Projects/Domain") | ||
case .Data: | ||
return .project(target: self.rawValue, path: "Projects/Data") | ||
case .DI: | ||
return .project(target: self.rawValue, path: "Projects/DI") | ||
case .DesignSystem: | ||
return .project(target: self.rawValue, path: "Projects/DesignSystem") | ||
default: | ||
return .project(target: "none", path: "") | ||
public extension DependencyInformation { | ||
static func dependencies(of name: String) -> [TargetDependency] { | ||
guard let name = DependencyInformation(rawValue: name) else { return [] } | ||
guard let modules: [DependencyInformation] = dependencyInfo[name] else { return [] } | ||
|
||
return modules.map { module in | ||
let name = module.rawValue | ||
|
||
if externalDependency.contains(module) { | ||
return .external(name: name) | ||
} else { | ||
return .project(target: name, path: .relativeToRoot("Projects/\(name)")) | ||
} | ||
Comment on lines
+31
to
+42
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. 더 깔끔해지고 모듈 확장성이 좋아졌네요! 👍👍 |
||
} | ||
} | ||
} |
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.
해당 Struct는 어떤 용도로 사용하는 걸까요??
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.
이거 프리뷰 테스트 용으로 만들었던 것 같습니다... (?) 발견 감사해요 수정해놓을게요!