Skip to content

Csi-powermax/Add Makefile Targets for Local PR Checks#433

Merged
santhoshatdell merged 2 commits intomainfrom
mg/act-pmax
Feb 28, 2025
Merged

Csi-powermax/Add Makefile Targets for Local PR Checks#433
santhoshatdell merged 2 commits intomainfrom
mg/act-pmax

Conversation

@mgandharva
Copy link
Contributor

Description

This PR adds Makefile targets to run GitHub Actions checks locally:

actions Target :
Runs all pull request checks with enhanced logging.
action-help Target :
Displays instructions for running specific workflows with act.
Usage:

Run all PR checks: make actions
Get workflow instructions: make action-help
Enhances local testing and debugging of GitHub Actions.

GitHub Issues

List the GitHub issues impacted by this PR:

GitHub Issue #
https://github.com/dell/csm/issues/1490

Checklist:

  • I have performed a self-review of my own code to ensure there are no formatting, vetting, linting, or security issues
  • I have verified that new and existing unit tests pass locally with my changes
  • I have not allowed coverage numbers to degenerate
  • I have maintained at least 90% code coverage
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my fix is effective or that my feature works
  • Backward compatibility is not broken

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Please also list any relevant details for your test configuration

  • Tested locally

@mgandharva mgandharva changed the title Add Makefile Targets for Local PR Checks Csi-powermax/Add Makefile Targets for Local PR Checks Jan 30, 2025
@santhoshatdell santhoshatdell merged commit 47e1e0d into main Feb 28, 2025
6 checks passed
@santhoshatdell santhoshatdell deleted the mg/act-pmax branch February 28, 2025 16:08
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