Skip to content

Commit

Permalink
Merge pull request #2634 from github/angelapwen/stop-using-artifact-flag
Browse files Browse the repository at this point in the history
Remove `artifact_v4_upgrade` feature flag logic and related PR checks
  • Loading branch information
angelapwen authored Dec 6, 2024
2 parents 3096afe + 0bb8007 commit 6f9e628
Show file tree
Hide file tree
Showing 21 changed files with 22 additions and 228 deletions.
1 change: 0 additions & 1 deletion .github/workflows/debug-artifacts-failure.yml
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,6 @@
name: PR Check - Debug artifacts after failure
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
CODEQL_ACTION_ARTIFACT_V4_UPGRADE: true
on:
push:
branches:
Expand Down
99 changes: 0 additions & 99 deletions .github/workflows/debug-artifacts-legacy.yml

This file was deleted.

1 change: 0 additions & 1 deletion .github/workflows/debug-artifacts.yml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@
name: PR Check - Debug artifact upload
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
CODEQL_ACTION_ARTIFACT_V4_UPGRADE: true
on:
push:
branches:
Expand Down
6 changes: 1 addition & 5 deletions lib/analyze-action-post.js

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

2 changes: 1 addition & 1 deletion lib/analyze-action-post.js.map

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

19 changes: 7 additions & 12 deletions lib/debug-artifacts.js

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

Loading

0 comments on commit 6f9e628

Please sign in to comment.