-
Notifications
You must be signed in to change notification settings - Fork 0
ci: Add initial action configuration and tests #3
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.
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
Conversation
Signed-off-by: Roger Barker <roger.barker@swirldslabs.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
This PR significantly expands the GitHub Action to support multiple programming languages and build tools, adding comprehensive test coverage and modernizing the configuration. The changes introduce support for Gradle, Rust, and Swift alongside the existing Node.js, Python, Java, and Go toolchains.
Key changes:
- Added support for Gradle, Rust, and Swift toolchain setup with configurable options
- Expanded inputs for advanced configuration (fetch-depth, caching options, architecture)
- Introduced comprehensive outputs for all toolchains to enable downstream workflow integration
- Restructured test workflow with matrix testing across Ubuntu, Windows, and macOS
- Added Harden Runner security step to audit egress network traffic
- Replaced README with placeholder indicating forthcoming documentation update
Reviewed Changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 16 comments.
| File | Description |
|---|---|
| action.yml | Major expansion of composite action adding Gradle, Rust, and Swift support with new inputs, outputs, and security hardening. Contains several bugs related to missing input mappings and inconsistent defaults. |
| .github/workflows/test.yml | Complete restructure of test workflow with matrix testing and dedicated jobs per toolchain. Multiple jobs are missing required checkout steps that will cause failures. |
| README.md | Replaced comprehensive documentation with "TBD" placeholder, temporarily removing all usage examples and input/output documentation. |
| .gitignore | Added extensive autogenerated gitignore templates covering Maven, Gradle, Node, Python, Java, Kotlin, Rust, and various IDEs for broader development environment support. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Co-authored-by: Andrew Brandt <andrew.brandt@hashgraph.com> Signed-off-by: Roger Barker <roger.barker@swirldslabs.com>
Signed-off-by: Roger Barker <roger.barker@swirldslabs.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull Request Overview
Copilot reviewed 3 out of 4 changed files in this pull request and generated 6 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Roger Barker <roger.barker@swirldslabs.com>
Signed-off-by: Roger Barker <roger.barker@swirldslabs.com>
Signed-off-by: Roger Barker <roger.barker@swirldslabs.com>
Signed-off-by: Roger Barker <roger.barker@swirldslabs.com>
Signed-off-by: Roger Barker <roger.barker@swirldslabs.com>
Signed-off-by: Roger Barker <roger.barker@swirldslabs.com>
Signed-off-by: Roger Barker <roger.barker@swirldslabs.com>
Signed-off-by: Roger Barker <roger.barker@swirldslabs.com>
Signed-off-by: Roger Barker <roger.barker@swirldslabs.com>
Signed-off-by: Roger Barker <roger.barker@swirldslabs.com>
Signed-off-by: Roger Barker <roger.barker@swirldslabs.com>
Signed-off-by: Roger Barker <roger.barker@swirldslabs.com>
Signed-off-by: Roger Barker <roger.barker@swirldslabs.com>
Signed-off-by: Roger Barker <roger.barker@swirldslabs.com>
Signed-off-by: Roger Barker <roger.barker@swirldslabs.com>
Description
This pull request significantly enhances the GitHub Actions workflow and documentation for the composite action. The workflow in
.github/workflows/test.ymlis refactored to provide comprehensive, modular testing for each supported language and tool, adds security hardening, and improves output verification. TheREADME.mdis rewritten for clarity, completeness, and up-to-date usage instructions, including detailed input/output tables and new examples.Workflow improvements and modularization:
.github/workflows/test.ymlto split testing into separate jobs for each supported language/tool (Node.js, Python, Java, Gradle, Go, Rust, Swift, and repository checkout), enabling more granular and maintainable CI. Each job now verifies relevant outputs for its setup.converted_to_draft,ready_for_review, etc.) and removed less relevant branches from triggers.Documentation and usage clarity:
README.mdto provide a clearer project description, highlight security features, and enumerate all supported languages and tools.These changes make the composite action easier to adopt, more secure, and much more maintainable for future development.
References: [1] [2]
Related Issue(s)
Closes #2