Skip to content

Commit bde5694

Browse files
henrymercerhenrymercer
committed
Comment why we don't run multiple autobuilders for other languages
1 parent 40e0374 commit bde5694

3 files changed

Lines changed: 15 additions & 1 deletion

File tree

‎lib/autobuild.js‎

Lines changed: 7 additions & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎lib/autobuild.js.map‎

Lines changed: 1 addition & 1 deletion
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎src/autobuild.ts‎

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -72,6 +72,13 @@ export async function determineAutobuildLanguages(
7272

7373
logger.debug(`Will autobuild ${languages.join(" and ")}.`);
7474

75+
// In general the autobuilders for other traced languages may conflict with
76+
// each other. Therefore if a user has requested more than one non-Go traced
77+
// language, we ask for manual build steps.
78+
// Matrixing the build would also work, but that would change the SARIF
79+
// categories, potentially leading to a "stale tips" situation where alerts
80+
// that should be fixed remain on a repo since they are linked to SARIF
81+
// categories that are no longer updated.
7582
if (autobuildLanguagesNoGo.length > 1) {
7683
logger.warning(
7784
`We will only automatically build ${languages.join(

0 commit comments

Comments
 (0)