Skip to content

Action.yml improvements #82

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 30 commits into from
Jan 18, 2025
Merged

Action.yml improvements #82

merged 30 commits into from
Jan 18, 2025
+59 −10

Conversation

@goergisn
Copy link
Contributor

@goergisn goergisn commented Jan 15, 2025 •
edited
Loading

Summary

  • Compiling binary CLI
  • Caching/Restoring binary CLI
  • Making workflow succeed on PRs from Forks
    • Adding Info regarding PRs from Forks to Github Step Summary

@goergisn goergisn changed the title using local action + resolved a warning Resolved a warning Jan 15, 2025
@goergisn goergisn changed the base branch from main to swift-package-improvements January 18, 2025 12:12
@goergisn goergisn changed the title Resolved a warning Using binary CLI + Caching Jan 18, 2025
Base automatically changed from swift-package-improvements to main January 18, 2025 12:17
@goergisn goergisn marked this pull request as draft January 18, 2025 12:17
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jan 18, 2025

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@goergisn goergisn changed the title Using binary CLI + Caching Action.yml improvements Jan 18, 2025
@github-actions
Copy link

github-actions bot commented Jan 18, 2025

✅ No changes detected

Comparing small-improvements to main

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

@goergisn goergisn marked this pull request as ready for review January 18, 2025 15:06
@goergisn goergisn merged commit 2cd4716 into main Jan 18, 2025
6 checks passed
@goergisn goergisn deleted the small-improvements branch January 18, 2025 15:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@MrSkwiggs MrSkwiggs MrSkwiggs approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@goergisn @MrSkwiggs