@@ -72,43 +72,45 @@ jobs:
7272 const prUrl = pr.html_url;
7373 console.log(`Found PR #${prNumber} for branch ${branch}: https://github.com/${context.repo.owner}/${context.repo.repo}/pull/${prNumber}`);
7474
75- // Check if PR is in draft state
76- if (pr.draft) {
77- core.setFailed(`PR #${prNumber} is in draft state. Please mark it as ready for review before deploying.`);
78- return;
79- }
80-
81- // Get detailed PR information including mergeable state
82- const { data: prDetails } = await github.rest.pulls.get({
83- owner: context.repo.owner,
84- repo: context.repo.repo,
85- pull_number: prNumber
86- });
87-
88- // Check if PR is mergeable and all requirements are satisfied
89- if (prDetails.mergeable === false) {
90- core.setFailed(`PR #${prNumber} is not in a mergeable state. Please resolve conflicts before deploying.`);
91- return;
92- }
93-
94- // The mergeable_state can be one of: clean, dirty, blocked, unstable, or unknown
95- // Only 'clean' means all requirements are met (checks passed, approvals received, no conflicts)
96- if (prDetails.mergeable_state !== 'clean') {
97- // Get more details about why it's not clean
98- let reason = '';
75+ if (false) {
76+ // Check if PR is in draft state
77+ if (pr.draft) {
78+ core.setFailed(`PR #${prNumber} is in draft state. Please mark it as ready for review before deploying.`);
79+ return;
80+ }
9981
100- if (prDetails.mergeable_state === 'blocked') {
101- reason = 'required checks or approvals are missing';
102- } else if (prDetails.mergeable_state === 'dirty') {
103- reason = 'there are merge conflicts';
104- } else if (prDetails.mergeable_state === 'unstable') {
105- reason = 'required checks are failing';
106- } else {
107- reason = `the mergeable state is "${prDetails.mergeable_state}"`;
82+ // Get detailed PR information including mergeable state
83+ const { data: prDetails } = await github.rest.pulls.get({
84+ owner: context.repo.owner,
85+ repo: context.repo.repo,
86+ pull_number: prNumber
87+ });
88+
89+ // Check if PR is mergeable and all requirements are satisfied
90+ if (prDetails.mergeable === false) {
91+ core.setFailed(`PR #${prNumber} is not in a mergeable state. Please resolve conflicts before deploying.`);
92+ return;
10893 }
10994
110- core.setFailed(`PR #${prNumber} is not ready to merge: ${reason}. Please resolve all issues before deploying.`);
111- return;
95+ // The mergeable_state can be one of: clean, dirty, blocked, unstable, or unknown
96+ // Only 'clean' means all requirements are met (checks passed, approvals received, no conflicts)
97+ if (prDetails.mergeable_state !== 'clean') {
98+ // Get more details about why it's not clean
99+ let reason = '';
100+
101+ if (prDetails.mergeable_state === 'blocked') {
102+ reason = 'required checks or approvals are missing';
103+ } else if (prDetails.mergeable_state === 'dirty') {
104+ reason = 'there are merge conflicts';
105+ } else if (prDetails.mergeable_state === 'unstable') {
106+ reason = 'required checks are failing';
107+ } else {
108+ reason = `the mergeable state is "${prDetails.mergeable_state}"`;
109+ }
110+
111+ core.setFailed(`PR #${prNumber} is not ready to merge: ${reason}. Please resolve all issues before deploying.`);
112+ return;
113+ }
112114 }
113115
114116 console.log(`PR #${prNumber} is in a clean mergeable state. All requirements satisfied. Proceeding with beta deployment.`);
0 commit comments