Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

mod: スキャン対象のディレクトリが存在するかどうかの条件分岐を追加 #8

Merged
merged 1 commit into from
Dec 21, 2023

Conversation

mutao-net
Copy link
Contributor

@mutao-net mutao-net commented Dec 21, 2023

修正内容

  • ディレクトリごと削除するPRを作成した場合、エラーが発生してしまうので修正します。
    • git diff で検査対象のPATHを指定している。
    • そのためディレクトリ毎削除した場合でも、検査対象のPATHとなるがPR BranchにはそのPATHが存在しない状況となっているため、スキャンで失敗するというもの。
  • https://github.com/globis-org/unlimited-front/actions/runs/7285531351/job/19852618893

動作確認①

github-action-test という repo を作成してそちらで検証しました。

  1. github-action-test repo で dir/dir2/hoge.txt という階層にファイルを用意
  2. dir/dir2 というディレクトリを削除するPRを作成 skipされることを確認する

image

動作確認②

  1. 動作確認①に加えて、dummyのtokenが記載された dummy.txt を修正する。
  2. dummy.txt の内容が検知されることを確認する。

image

@mutao-net mutao-net requested a review from a team as a code owner December 21, 2023 09:17
@mutao-net mutao-net force-pushed the add/workdirs-check-step branch from 380e7fe to b6817fa Compare December 21, 2023 09:44
Copy link
Member

@chroju chroju left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

これだとファイル削除のみでディレクトリはまだ残っている場合、
変更を加えていない、同ディレクトリ配下の残存ファイルがスキャン対象になりますが、
そこはまぁ、許容ですかね。。

@mutao-net mutao-net merged commit 25c48cc into main Dec 21, 2023
4 checks passed
mutao-net added a commit that referenced this pull request Dec 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants