Skip to content

feature/release-creator#112

Merged
shaynafinocchiaro merged 26 commits into
mainfrom
feature/release-creator
Jan 30, 2025
Merged

feature/release-creator#112
shaynafinocchiaro merged 26 commits into
mainfrom
feature/release-creator

Conversation

@harishp8889

@harishp8889 harishp8889 commented Jan 24, 2025

Copy link
Copy Markdown
Contributor

Description

  1. Added support for releasing multiple images
  2. Modularized release creator action for reusability
  3. Updated input type for version and images by taking it automatically instead of manual version inputs
  4. Added binary build support

Issues

List the issues impacted by this PR:

Issue ID
https://github.com/dell/csm/issues/1490

Checklist:

  • I have performed a self-review of my own code
  • 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

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

@harishp8889 harishp8889 self-assigned this Jan 24, 2025
Comment thread .github/CODEOWNERS
@gallacher

Copy link
Copy Markdown
Collaborator

@harishp8889, can you please list out all the test scenarios you tested here.

Comment thread .github/workflows/csm-release-driver-module.yaml Outdated
Comment thread .github/workflows/csm-release-driver-module.yaml Outdated
Comment thread .github/workflows/release-creator.yaml Outdated
Comment thread .github/workflows/csm-release-libs.yaml Outdated
Comment thread .github/workflows/release-creator.yaml

@meggm meggm left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

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.

5 participants