Skip to content

Commit e2c807f

Browse files
authored
Merge pull request Expensify#71504 from Expensify/jules-removeCherryPickQaLine
2 parents 13a7188 + 6c839db commit e2c807f

3 files changed

Lines changed: 3 additions & 16 deletions

File tree

.github/actions/javascript/markPullRequestsAsDeployed/index.js

Lines changed: 1 addition & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -12845,7 +12845,7 @@ async function run() {
1284512845
const webResult = getDeployTableMessage(core.getInput('WEB', { required: true }));
1284612846
const date = core.getInput('DATE');
1284712847
const note = core.getInput('NOTE');
12848-
function getDeployMessage(deployer, deployVerb, prTitle) {
12848+
function getDeployMessage(deployer, deployVerb) {
1284912849
let message = `🚀 [${deployVerb}](${workflowURL}) to ${isProd ? 'production' : 'staging'}`;
1285012850
message += ` by https://github.com/${deployer} in version: ${version} `;
1285112851
if (date) {
@@ -12855,11 +12855,6 @@ async function run() {
1285512855
message += `\n\nplatform | result\n---|---\n🖥 desktop 🖥|${desktopResult}`;
1285612856
message += `\n🕸 web 🕸|${webResult}`;
1285712857
message += `\n🤖 android 🤖|${androidResult}\n🍎 iOS 🍎|${iOSResult}`;
12858-
if (deployVerb === 'Cherry-picked' && !/no ?qa/gi.test(prTitle ?? '')) {
12859-
// eslint-disable-next-line max-len
12860-
message +=
12861-
'\n\n@Expensify/applauseleads please QA this PR and check it off on the [deploy checklist](https://github.com/Expensify/App/issues?q=is%3Aopen+is%3Aissue+label%3AStagingDeployCash) if it passes.';
12862-
}
1286312858
if (note) {
1286412859
message += `\n\n_Note:_ ${note}`;
1286512860
}

.github/actions/javascript/markPullRequestsAsDeployed/markPullRequestsAsDeployed.ts

Lines changed: 1 addition & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -110,7 +110,7 @@ async function run() {
110110
const date = core.getInput('DATE');
111111
const note = core.getInput('NOTE');
112112

113-
function getDeployMessage(deployer: string, deployVerb: string, prTitle?: string): string {
113+
function getDeployMessage(deployer: string, deployVerb: string): string {
114114
let message = `🚀 [${deployVerb}](${workflowURL}) to ${isProd ? 'production' : 'staging'}`;
115115
message += ` by https://github.com/${deployer} in version: ${version} `;
116116
if (date) {
@@ -121,12 +121,6 @@ async function run() {
121121
message += `\n🕸 web 🕸|${webResult}`;
122122
message += `\n🤖 android 🤖|${androidResult}\n🍎 iOS 🍎|${iOSResult}`;
123123

124-
if (deployVerb === 'Cherry-picked' && !/no ?qa/gi.test(prTitle ?? '')) {
125-
// eslint-disable-next-line max-len
126-
message +=
127-
'\n\n@Expensify/applauseleads please QA this PR and check it off on the [deploy checklist](https://github.com/Expensify/App/issues?q=is%3Aopen+is%3Aissue+label%3AStagingDeployCash) if it passes.';
128-
}
129-
130124
if (note) {
131125
message += `\n\n_Note:_ ${note}`;
132126
}

tests/unit/markPullRequestsAsDeployedTest.ts

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -284,9 +284,7 @@ platform | result
284284
🖥 desktop 🖥|success ✅
285285
🕸 web 🕸|success ✅
286286
🤖 android 🤖|success ✅
287-
🍎 iOS 🍎|success ✅
288-
289-
@Expensify/applauseleads please QA this PR and check it off on the [deploy checklist](https://github.com/Expensify/App/issues?q=is%3Aopen+is%3Aissue+label%3AStagingDeployCash) if it passes.`,
287+
🍎 iOS 🍎|success ✅`,
290288
issue_number: 3,
291289
owner: CONST.GITHUB_OWNER,
292290
repo: CONST.APP_REPO,

0 commit comments

Comments
 (0)