Skip to content

Simplify PR number retrieval in deploy workflow#84

Merged
hshah-mitre merged 5 commits into
mainfrom
hshah-mitre-patch-1
Dec 23, 2025
Merged

Simplify PR number retrieval in deploy workflow#84
hshah-mitre merged 5 commits into
mainfrom
hshah-mitre-patch-1

Conversation

@hshah-mitre

Copy link
Copy Markdown
Collaborator

Removed redundant PR number retrieval logic and fallback mechanism.

Description

This merge request addresses, and describe the problem or user story being addressed.

Changes Made

Provide code snippets or screenshots as needed.

Related Issues

Provide links to the related issues or feature requests.

Additional Notes

Include any extra information or considerations for reviewers, such as impacted areas of the codebase.

Merge Request Checklists

  • Documentation reflects the changes made.
  • I have already covered the unit testing.

Preview

👀 Preview your changes here (replace <PR_NUMBER> with your PR number):

http://ai4sdlcmd.ai4sdlc.c3ib.org/MergeRequest/<PR_NUMBER>/index.html


Assign to me - contributors

Assignees will be assigned to me.

Reviewers/Approvals

Reviewers will be assigned to the ai4sdlc-reviewers group.

Final Reviewers/Merger

Final approvals will be assigned to the ai4sdlc-approval group.

Removed redundant PR number retrieval logic and fallback mechanism.
@github-actions

Copy link
Copy Markdown

Please review the changes at the following site: http://ai4sdlcmd.ai4sdlc.c3ib.org/mergeRequest/84/index.html

@hshah-mitre hshah-mitre requested a review from a team December 23, 2025 23:10
Commented out the condition for running deployment on the main branch and added debug echo statements for PR number.
@github-actions

Copy link
Copy Markdown

Please review the changes at the following site: http://ai4sdlcmd.ai4sdlc.c3ib.org/mergeRequest/84/index.html

Updated the way PR_NUMBER is retrieved and echoed in the deploy workflow.
@github-actions

Copy link
Copy Markdown

Please review the changes at the following site: http://ai4sdlcmd.ai4sdlc.c3ib.org/mergeRequest/84/index.html

@github-actions

Copy link
Copy Markdown

Please review the changes at the following site: http://ai4sdlcmd.ai4sdlc.c3ib.org/mergeRequest/84/index.html

@github-actions

Copy link
Copy Markdown

Please review the changes at the following site: http://ai4sdlcmd.ai4sdlc.c3ib.org/mergeRequest/84/index.html

@Conwenu Conwenu left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

5:42

@github-actions

Copy link
Copy Markdown

@Code-dot-mil/ai4sdlc-approval PR #84 has a new review comment from @Conwenu and is ready for final review.

@hshah-mitre hshah-mitre requested a review from a team December 23, 2025 23:42
@hshah-mitre hshah-mitre added this pull request to the merge queue Dec 23, 2025
Merged via the queue into main with commit e6ceffd Dec 23, 2025
15 checks passed
@hshah-mitre hshah-mitre deleted the hshah-mitre-patch-1 branch December 23, 2025 23:43
@github-actions

Copy link
Copy Markdown

Please review the changes at the following site: http://ai4sdlcmd.ai4sdlc.c3ib.org/mergeRequest/84/index.html

@hshah-mitre hshah-mitre requested a review from a team December 23, 2025 23:43
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.

2 participants