Skip to content

Added support for patch release and license header check#145

Merged
harishp8889 merged 17 commits intomainfrom
bugfix/patch-release
Apr 15, 2025
Merged

Added support for patch release and license header check#145
harishp8889 merged 17 commits intomainfrom
bugfix/patch-release

Conversation

@harishp8889
Copy link
Copy Markdown
Contributor

@harishp8889 harishp8889 commented Apr 8, 2025

Description

  1. Added support for patch release (n-1, n-2)
  2. Added support for license header validation

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 Apr 8, 2025
@harishp8889 harishp8889 changed the title Added support for patch release (n-1, n-2) Added support for patch release and license header check Apr 8, 2025
Comment thread .github/configs/.licenserc.yaml Outdated
meggm
meggm previously approved these changes Apr 10, 2025
Copy link
Copy Markdown
Contributor

@meggm meggm left a comment

Choose a reason for hiding this comment

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

LGTM

donatwork
donatwork previously approved these changes Apr 10, 2025
khareRajshree
khareRajshree previously approved these changes Apr 10, 2025
Comment thread .github/workflows/check-license-header.yaml
Comment thread .github/workflows/csm-release-driver-module.yaml
Comment thread .github/workflows/release-creator.yaml
Comment thread .github/configs/.licenserc.yaml Outdated
Comment thread .github/configs/.licenserc.yaml Outdated
@harishp8889 harishp8889 dismissed stale reviews from khareRajshree, donatwork, and meggm via 2c5d3cb April 10, 2025 17:02
Comment thread .github/workflows/check-license-header.yaml Outdated
Comment thread .github/workflows/check-license-header.yaml Outdated
Comment thread .github/workflows/check-license-header.yaml Outdated
Comment thread .github/configs/.licenserc.yaml
@harishp8889 harishp8889 merged commit 6f2e6d6 into main Apr 15, 2025
2 checks passed
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.

6 participants