Merge pull request #3385 from github/henrymercer/fix-ff-lookup · github/codeql-action@66bcc86 · GitHub

github / codeql-action Public

• Notifications You must be signed in to change notification settings
• Fork 487
• Star 1.6k

• Code
• Issues 158
• Pull requests 35
• Actions
• Security and quality 2
• Insights

Additional navigation options

Commit 66bcc86

Browse filesBrowse the repository at this point in the historyBrowse files

henrymercer

authored

Merge pull request #3385 from github/henrymercer/fix-ff-lookup

Fix feature flag lookup when uploading DB

2 parents 0d648eb + 44e589b commit 66bcc86
Copy full SHA for 66bcc86

2 files changed

+2-2Lines changed: 2 additions & 2 deletions

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• lib
  • analyze-action.js
• src
  • database-upload.ts

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎lib/analyze-action.js‎

Copy file name to clipboardExpand all lines: lib/analyze-action.js

+1-1Lines changed: 1 addition & 1 deletion

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

Collapse file

‎src/database-upload.ts‎

Copy file name to clipboardExpand all lines: src/database-upload.ts

+1-1Lines changed: 1 addition & 1 deletion

Original file line number	Diff line number	Diff line change
@@ -70,7 +70,7 @@ export async function cleanupAndUploadDatabases(
70	70	// If config.overlayDatabaseMode is OverlayBase, then we have overlay base databases for all languages.
71	71	const shouldUploadOverlayBase =
72	72	config.overlayDatabaseMode === OverlayDatabaseMode.OverlayBase &&
73		- (await features.getValue(Feature.UploadOverlayDbToApi));
	73	+ (await features.getValue(Feature.UploadOverlayDbToApi, codeql));
74	74	const cleanupLevel = shouldUploadOverlayBase
75	75	? CleanupLevel.Overlay
76	76	: CleanupLevel.Clear;

0 commit comments

Comments

0 (0)