Skip to content

Commit b4e0d87

Browse files
committed
swift-package-migrate: Don't end logging messages with a period
Other commands don't do this. Be consistent.
1 parent a61fe0e commit b4e0d87

File tree

1 file changed

+14
-6
lines changed

1 file changed

+14
-6
lines changed

Sources/Commands/PackageCommands/Migrate.swift

Lines changed: 14 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -94,7 +94,7 @@ extension SwiftPackageCommand {
9494
// Next, let's build all of the individual targets or the
9595
// whole project to get diagnostic files.
9696

97-
print("> Starting the build.")
97+
print("> Starting the build")
9898
if let targets = self.options.targets {
9999
for target in targets {
100100
try await buildSystem.build(subset: .target(target))
@@ -123,7 +123,7 @@ extension SwiftPackageCommand {
123123
// If the build suceeded, let's extract all of the diagnostic
124124
// files from build plan and feed them to the fix-it tool.
125125

126-
print("> Applying fix-its.")
126+
print("> Applying fix-its")
127127

128128
var summary = SwiftFixIt.Summary(numberOfFixItsApplied: 0, numberOfFilesChanged: 0)
129129
let fixItDuration = try ContinuousClock().measure {
@@ -155,17 +155,17 @@ extension SwiftPackageCommand {
155155
fractionalPart: .init(lengthLimits: 0 ... 3, roundingRule: .up)
156156
)
157157
)
158-
message += ")."
158+
message += ")"
159159

160160
print(message)
161161
}
162162

163163
// Once the fix-its were applied, it's time to update the
164164
// manifest with newly adopted feature settings.
165165

166-
print("> Updating manifest.")
166+
print("> Updating manifest")
167167
for module in modules.map(\.module) {
168-
swiftCommandState.observabilityScope.emit(debug: "Adding feature(s) to '\(module.name)'.")
168+
swiftCommandState.observabilityScope.emit(debug: "Adding feature(s) to '\(module.name)'")
169169
try self.updateManifest(
170170
for: module.name,
171171
add: features,
@@ -235,7 +235,15 @@ extension SwiftPackageCommand {
235235
verbose: !self.globalOptions.logging.quiet
236236
)
237237
} catch {
238-
swiftCommandState.observabilityScope.emit(error: "Could not update manifest for '\(target)' (\(error)). Please enable '\(try features.map { try $0.swiftSettingDescription }.joined(separator: ", "))' features manually.")
238+
try swiftCommandState.observabilityScope.emit(
239+
error: """
240+
Could not update manifest for '\(target)' (\(error)). \
241+
Please enable '\(
242+
features.map { try $0.swiftSettingDescription }
243+
.joined(separator: ", ")
244+
)' features manually
245+
"""
246+
)
239247
}
240248
}
241249

0 commit comments

Comments
 (0)