Skip to content

Conversation

@goergisn
Copy link
Contributor

@goergisn goergisn commented Jan 2, 2025

Summary

  • Renaming Change.ChangeType.change to Change.ChangeType.modification
  • Adding changes breakdown to markdown output header

Fixed issue: #76


The new header looks like this now if there are potentially breaking changes


⚠️ 45 public changes detected ⚠️

Comparing new_public to old_public

❇️28 Additions
🔀15 Modifications
2 Removals

ModuleName

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 2, 2025

@github-actions
Copy link

github-actions bot commented Jan 2, 2025

⚠️ 4 public changes detected ⚠️

Comparing change-summary to main

❇️2 Additions
2 Removals

PADCore

Change.ChangeType

❇️ Added

case modification(
  oldDescription: Swift.String,
  newDescription: Swift.String
)
public var isModification: Swift.Bool { get }

❌ Removed

case change(
  oldDescription: Swift.String,
  newDescription: Swift.String
)
public var isChange: Swift.Bool { get }

Analyzed targets: FileHandlingModule, PADCore, PADLogging, PADOutputGenerator, PADPackageFileAnalyzer, PADProjectBuilder, PADSwiftInterfaceDiff, PADSwiftInterfaceFileLocator, ShellModule, SwiftPackageFileHelperModule

@goergisn goergisn requested review from atmamont and nauaros January 2, 2025 13:45
@goergisn goergisn merged commit efac16c into main Jan 2, 2025
8 of 9 checks passed
@goergisn goergisn deleted the change-summary branch January 2, 2025 14:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants