Skip to content
Merged
Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
20 commits
Select commit Hold shift + click to select a range
5091e42
Overlay: Remove repository owner restriction
kaspersv Nov 13, 2025
497c7f6
Update changelog and version after v4.31.3
github-actions[bot] Nov 13, 2025
246edb9
Rebuild
github-actions[bot] Nov 13, 2025
86b7d4f
Merge pull request #3294 from github/mergeback/v4.31.3-to-main-014f16e7
mbg Nov 13, 2025
85f1517
Merge pull request #3285 from github/kaspersv/remove-overlay-org-rest…
kaspersv Nov 14, 2025
b9620e1
Bump js-yaml from 4.1.0 to 4.1.1
dependabot[bot] Nov 15, 2025
8c254d0
Rebuild
github-actions[bot] Nov 15, 2025
c1a2b73
Merge pull request #3301 from github/dependabot/npm_and_yarn/js-yaml-…
mbg Nov 16, 2025
ed3a013
Change v3 deprecation message to warning.
mario-campos Nov 17, 2025
023fd08
Add CHANGELOG.md entry for "v3 deprecation" to warning change.
mario-campos Nov 17, 2025
fc329e3
Revert "Add CHANGELOG.md entry for "v3 deprecation" to warning change."
mario-campos Nov 17, 2025
c418a0f
Bump ruby/setup-ruby
dependabot[bot] Nov 17, 2025
e546fff
Rebuild
github-actions[bot] Nov 17, 2025
07eae64
Merge pull request #3303 from github/mario-campos/v3-core-warning
mario-campos Nov 17, 2025
7bcdb4b
Add additional options to PR template and clarify some
mbg Nov 17, 2025
ffa63f0
Merge pull request #3307 from github/dependabot/github_actions/dot-gi…
mbg Nov 17, 2025
de12435
Merge pull request #3308 from github/mbg/pr-template/nov25
mbg Nov 18, 2025
528362a
Bump `glob` to at least `11.1.0`
mbg Nov 18, 2025
70434f6
Merge pull request #3311 from github/mbg/deps/bump-glob
mbg Nov 18, 2025
c9cb6f9
Update changelog for v4.31.4
github-actions[bot] Nov 18, 2025
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
9 changes: 2 additions & 7 deletions lib/init-action.js

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

43 changes: 2 additions & 41 deletions src/config-utils.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -990,7 +990,6 @@ interface OverlayDatabaseModeTestSetup {
features: Feature[];
isPullRequest: boolean;
isDefaultBranch: boolean;
repositoryOwner: string;
buildMode: BuildMode | undefined;
languages: Language[];
codeqlVersion: string;
Expand All @@ -1003,7 +1002,6 @@ const defaultOverlayDatabaseModeTestSetup: OverlayDatabaseModeTestSetup = {
features: [],
isPullRequest: false,
isDefaultBranch: false,
repositoryOwner: "github",
buildMode: BuildMode.None,
languages: [KnownLanguage.javascript],
codeqlVersion: CODEQL_OVERLAY_MINIMUM_VERSION,
Expand Down Expand Up @@ -1049,12 +1047,6 @@ const getOverlayDatabaseModeMacro = test.macro({
.stub(actionsUtil, "isAnalyzingPullRequest")
.returns(setup.isPullRequest);

// Mock repository owner
const repository = {
owner: setup.repositoryOwner,
repo: "test-repo",
};

// Set up CodeQL mock
const codeql = mockCodeQLVersion(setup.codeqlVersion);

Expand All @@ -1077,7 +1069,6 @@ const getOverlayDatabaseModeMacro = test.macro({

const result = await configUtils.getOverlayDatabaseMode(
codeql,
repository,
features,
setup.languages,
tempDir, // sourceRoot
Expand Down Expand Up @@ -1499,23 +1490,9 @@ test(

test(
getOverlayDatabaseModeMacro,
"Overlay PR analysis by env for dsp-testing",
{
overlayDatabaseEnvVar: "overlay",
repositoryOwner: "dsp-testing",
},
{
overlayDatabaseMode: OverlayDatabaseMode.Overlay,
useOverlayDatabaseCaching: false,
},
);

test(
getOverlayDatabaseModeMacro,
"Overlay PR analysis by env for other-org",
"Overlay PR analysis by env",
{
overlayDatabaseEnvVar: "overlay",
repositoryOwner: "other-org",
},
{
overlayDatabaseMode: OverlayDatabaseMode.Overlay,
Expand All @@ -1525,34 +1502,18 @@ test(

test(
getOverlayDatabaseModeMacro,
"Overlay PR analysis by feature flag for dsp-testing",
"Overlay PR analysis by feature flag",
{
languages: [KnownLanguage.javascript],
features: [Feature.OverlayAnalysis, Feature.OverlayAnalysisJavascript],
isPullRequest: true,
repositoryOwner: "dsp-testing",
},
{
overlayDatabaseMode: OverlayDatabaseMode.Overlay,
useOverlayDatabaseCaching: true,
},
);

test(
getOverlayDatabaseModeMacro,
"No overlay PR analysis by feature flag for other-org",
{
languages: [KnownLanguage.javascript],
features: [Feature.OverlayAnalysis, Feature.OverlayAnalysisJavascript],
isPullRequest: true,
repositoryOwner: "other-org",
},
{
overlayDatabaseMode: OverlayDatabaseMode.None,
useOverlayDatabaseCaching: false,
},
);

test(
getOverlayDatabaseModeMacro,
"Fallback due to autobuild with traced language",
Expand Down
9 changes: 0 additions & 9 deletions src/config-utils.ts
Original file line number Diff line number Diff line change
Expand Up @@ -579,17 +579,11 @@ const OVERLAY_ANALYSIS_CODE_SCANNING_FEATURES: Record<Language, Feature> = {
};

async function isOverlayAnalysisFeatureEnabled(
repository: RepositoryNwo,
features: FeatureEnablement,
codeql: CodeQL,
languages: Language[],
codeScanningConfig: UserConfig,
): Promise<boolean> {
// TODO: Remove the repository owner check once support for overlay analysis
// stabilizes, and no more backward-incompatible changes are expected.
if (!["github", "dsp-testing"].includes(repository.owner)) {
return false;
}
if (!(await features.getValue(Feature.OverlayAnalysis, codeql))) {
return false;
}
Expand Down Expand Up @@ -647,7 +641,6 @@ async function isOverlayAnalysisFeatureEnabled(
*/
export async function getOverlayDatabaseMode(
codeql: CodeQL,
repository: RepositoryNwo,
features: FeatureEnablement,
languages: Language[],
sourceRoot: string,
Expand Down Expand Up @@ -676,7 +669,6 @@ export async function getOverlayDatabaseMode(
);
} else if (
await isOverlayAnalysisFeatureEnabled(
repository,
features,
codeql,
languages,
Expand Down Expand Up @@ -846,7 +838,6 @@ export async function initConfig(
const { overlayDatabaseMode, useOverlayDatabaseCaching } =
await getOverlayDatabaseMode(
inputs.codeql,
inputs.repository,
inputs.features,
config.languages,
inputs.sourceRoot,
Expand Down
Loading