Skip to content

Commit bde5694

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

File tree

3 files changed

+15
-1
lines changed

3 files changed

+15
-1
lines changed

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)
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy