Skip to content

Commit

Permalink
Fix issue with .pytool/Plugin removal which was introduced with the l…
Browse files Browse the repository at this point in the history
…atest codeql.yml change. (#303)

The latest codeql.yml change updated the cleanup step to find
.pytool/Plugin folder directly instead of using a relevant path form the
CodeQL plugin directory.

That change didn't take into account how all branches from
release/202302 and older have the .pytool/Plugin version of CodeQL and
was deleting all plugins in .pytool besides CompilerPlugin. This change
excludes the CodeQL plugin if it exists as well.
  • Loading branch information
kenlautner authored Jan 24, 2024
1 parent e551a6b commit 45598bc
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions .sync/workflows/leaf/codeql.yml
Original file line number Diff line number Diff line change
Expand Up @@ -352,8 +352,8 @@ jobs:
import shutil
from pathlib import Path
# Only this plugin is needed for CodeQL
plugins_to_keep = ['CompilerPlugin']
# Only these two plugins are needed for CodeQL
plugins_to_keep = ['CodeQL', 'CompilerPlugin']
plugin_dir = Path(os.environ['PYTOOL_PLUGIN_DIR']).absolute()
if plugin_dir.is_dir():
Expand Down

0 comments on commit 45598bc

Please sign in to comment.