michael/swift-bump-version
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases 1 Wiki Activity

feat: Removes some old todo comments.

Browse Source
This commit is contained in:
Michael Housh
2024-12-26 14:04:37 -05:00
parent 2650bdc670
commit f118d2eaa6
2 changed files with 2 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

1
Sources/CliClient/Internal/CliClient+run.swift
View File

@@ -154,7 +154,6 @@ extension CliClient.SharedOptions {
let bumped = semvar.bump(type) let bumped = semvar.bump(type)
let version = bumped.versionString(withPreReleaseTag: allowPreReleaseTag) let version = bumped.versionString(withPreReleaseTag: allowPreReleaseTag)
// TODO: This doesn't work as expected w/o checking what the loaded semvar in the file is / was.
guard bumped != semvar || hasChanges else { guard bumped != semvar || hasChanges else {
logger.debug("No change, skipping.") logger.debug("No change, skipping.")
return return

2
Sources/ConfigurationClient/Configuration.swift
View File

@@ -1,6 +1,8 @@
import CustomDump import CustomDump
import Foundation import Foundation
// TODO: Add pre-release strategy that just bumps an integer.
/// Represents configuration that can be set via a file, generally in the root of the /// Represents configuration that can be set via a file, generally in the root of the
/// project directory. /// project directory.
/// ///