Skip to content

Adding change breakdown to markdown output #77

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Merged
merged 2 commits into from
Jan 2, 2025
Merged

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

Copy link

sonarqubecloud bot commented Jan 2, 2025

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 nauaros and atmamont 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
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants