-
Notifications
You must be signed in to change notification settings - Fork 13
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
[#494] Migrate make.sh to Swift command #504
Conversation
00d257d
to
7650872
Compare
14bbb59
to
77e08a0
Compare
Scripts/Swift/SetUpiOSProject.swift
Outdated
print("=> 🔎 Replacing package and package name within files...") | ||
|
||
try fileManager.replaceAllOccurrences(of: CONSTANT_BUNDLE_STAGING, to: bundleIdStaging) | ||
try fileManager.replaceAllOccurrences(of: CONSTANT_BUNDLE_PRODUCTION, to: bundleIdProduction) | ||
try fileManager.replaceAllOccurrences(of: CONSTANT_PROJECT_NAME, to: projectNameNoSpace) | ||
try fileManager.replaceAllOccurrences(of: CONSTANT_MINIMUM_VERSION, to: minimumVersion) | ||
print("✅ Completed") |
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.
Regarding the project info replacement, such as project name, package, app id, etc., we might consider using a Mustache template implementation in Swift https://github.com/groue/GRMustache.swift to reduce the boilerplate code and even make the core template to be more flexible with extra mustache supported rules 🤔
Just my suggestion for the next steps, not in this PR @nimblehq/ios-chapter. This is what we're doing in Flutter 😛
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.
@luongvo What do you use to generate folders and replace in files? The GRMustache.swift seems to only generate text based on the template.
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.
caf74b2
to
7e77550
Compare
67030aa
to
27a283c
Compare
7e77550
to
ad686eb
Compare
ad686eb
to
38eb870
Compare
38eb870
to
20759d8
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.
It's good to go 👍 Great job
make.sh
to Swift command #494What happened 👀
Migrate make.sh to Swift Command Line
Insight 📝
make.sh
for running the project with minimal change to command. Will be removed in Package all Swift Commands #495copy
andcreateDirectory
inFileManager
.~=
matcher toString
.SetUpCICDService
to struct to be called bySetUpIOSProject
.SetUpDeliveryConstants
to struct to be called bySetUpIOSProject
.SetUpInterface
to struct to be called bySetUpIOSProject
.SetUpIOSProject.swift
which will replacemake.sh
Proof Of Work 📹
Projects build successfully and CICD test works.