Skip to content

Conversation

@rahul-infra
Copy link
Contributor

No description provided.

@rahul-infra rahul-infra changed the title feat!: Updated pr title check and preview release type feat!: breaking change to workflow BREAKING CHANGE: Updated release preview mechanism Dec 2, 2025
@infraspecdev infraspecdev deleted a comment from github-actions bot Dec 2, 2025
@infraspecdev infraspecdev deleted a comment from github-actions bot Dec 2, 2025
@infraspecdev infraspecdev deleted a comment from github-actions bot Dec 2, 2025
@rahul-infra rahul-infra changed the title feat!: breaking change to workflow BREAKING CHANGE: Updated release preview mechanism fix: breaking change to workflow Dec 2, 2025
@rahul-infra rahul-infra force-pushed the fix/github-release-type branch from bbe9fdf to 6a2736a Compare December 2, 2025 09:55
@rahul-infra rahul-infra changed the title fix: breaking change to workflow feat!: Updated pr title check and preview release type Dec 2, 2025
@infraspecdev infraspecdev deleted a comment from github-actions bot Dec 2, 2025
@infraspecdev infraspecdev deleted a comment from github-actions bot Dec 2, 2025
@infraspecdev infraspecdev deleted a comment from github-actions bot Dec 2, 2025
run: |
# Get commits from this PR
COMMITS=$(git log origin/${{ github.base_ref }}..HEAD --pretty=format:"%s")
echo "Commits in this PR:"

Choose a reason for hiding this comment

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

if we are using semantic release to determine this, why are we adding our own script for version?
Why not use semantic release action itself?


body = `## Release Preview\n\n**Release Type:** \`${releaseType}\`\n**Next Version:** \`v${version}\`\n\n### Release Notes\n${notes}\n\n---\n*This shows what release will be created when this PR is merged.*`;
} else {
body = '## Release Preview\n\nNo new release will be created from this PR.\n\nThis PR does not contain conventional commits that trigger a release.';

Choose a reason for hiding this comment

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

would suggest using existing action instead of writing out own script. Please refer to how terraform output is commented on the PR.

@infraspecdev infraspecdev deleted a comment from github-actions bot Dec 2, 2025
@infraspecdev infraspecdev deleted a comment from github-actions bot Dec 2, 2025
Made changes in terraform workflow

Updated github workflows.

removed github token decleration in my new version preview

feat!: breaking change to workflow

BREAKING CHANGE: Updated release preview mechanism

fix: use PR merge ref for version preview

fix: changes versionpreview.yaml

fix: Updated Preview.yaml file

fix: updated versions-preview workflow making it edit the existing release preview

refactor: simplify version preview using semantic-release action

fix: configure semantic-release to analyze PR branches

fix: use semantic-release with proper branch configuration

fix: create branch instead of renaming in detached HEAD

debug: add logging to see what semantic-release sees

fix: pass PR number as input to version-preview workflow

fix: made changes in version-preivew for using sematic release.
@rahul-infra rahul-infra force-pushed the fix/github-release-type branch from 4a21396 to 5816a0d Compare December 2, 2025 11:01
@infraspecdev infraspecdev deleted a comment from github-actions bot Dec 2, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Dec 2, 2025

Release Preview

No new release will be created from this PR.

This PR does not contain conventional commits that trigger a release.

BREAKING CHANGES: Made changes for readme file in cross-account examples.
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.

3 participants