Skip to content

Commit 76b3b00

Browse files
committed
OB-10135: trim comments to one-liners
1 parent 518a61f commit 76b3b00

2 files changed

Lines changed: 2 additions & 8 deletions

File tree

bin/commands/runs.js

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -340,10 +340,7 @@ module.exports = function run(args, rawArgs) {
340340
utils.setProcessHooks(data.build_id, bsConfig, bs_local, args, buildReportData);
341341
if(isTestObservabilitySession) {
342342
utils.setO11yProcessHooks(data.build_id, bsConfig, bs_local, args, buildReportData);
343-
// OB-10135: cache the automate build id so `stopBuildUpstream`
344-
// can include it on the obs-api buildStop payload. Enables the
345-
// synergy dashboard to call the railsApp resource-errors
346-
// endpoint in the automate id-space directly.
343+
// OB-10135: read by stopBuildUpstream in testObservability/helper/helper.js
347344
process.env.BROWSERSTACK_AUTOMATION_BUILD_ID = data.build_id;
348345
}
349346
let message = `${data.message}! ${Constants.userMessages.BUILD_CREATED} with build id: ${data.build_id}`;

bin/testObservability/helper/helper.js

Lines changed: 1 addition & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -685,10 +685,7 @@ exports.stopBuildUpstream = async () => {
685685
} else {
686686
const data = {
687687
'stop_time': (new Date()).toISOString(),
688-
// OB-10135: propagate the automate build hashed_id (cached at
689-
// build-create time in runs.js) so obs-api can persist it and the
690-
// synergy dashboard can call the railsApp resource-errors endpoint
691-
// in the automate id-space directly.
688+
// OB-10135: cached in bin/commands/runs.js at build-create
692689
'automate_build_id': process.env.BROWSERSTACK_AUTOMATION_BUILD_ID
693690
};
694691
const config = {

0 commit comments

Comments
 (0)