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

feat: アスタリスクが入ったファイル名を拒否する #185

Closed
KisaragiEffective opened this issue May 13, 2022 · 6 comments · Fixed by #691
Closed

feat: アスタリスクが入ったファイル名を拒否する #185

KisaragiEffective opened this issue May 13, 2022 · 6 comments · Fixed by #691

Comments

@KisaragiEffective
Copy link
Contributor

Describe the solution you'd like (要望の内容)

  • GitHub Actionsでアスタリスクが入ったファイルが作成されたときRequest Changeする
    • Windowsでは正常に取り扱えない

Describe alternatives you've considered (代替え案)

  • 多分ない

Additional context (追加情報)

@KisaragiEffective KisaragiEffective self-assigned this May 13, 2022
@m1sk9 m1sk9 pinned this issue May 13, 2022
@KisaragiEffective
Copy link
Contributor Author

実装方針

  • git --list-changed | grep -Ec '^.*\*.*$'
  • if does not match, exit
  • use andrewmusgrave/[email protected] and REQUEST_CHANGES

@KisaragiEffective
Copy link
Contributor Author

現在においてのアーキテクチャ的な問題などによるリリースの支障ではないためP0を解除

@MikuroXina MikuroXina unpinned this issue Jun 22, 2022
@KisaragiEffective
Copy link
Contributor Author

KisaragiEffective commented Jul 7, 2022

現在においてのアーキテクチャ的な問題などによるリリースの支障ではないためP0を解除

@m2en 現状では非Windowsユーザー (例: ext4をデータパーティションとして使っているLinuxユーザー) が気をつければいいという話にとどまっているのでpriority:Criticalは不適当だと考える。

@KisaragiEffective KisaragiEffective removed their assignment Jul 7, 2022
@raiga0310
Copy link
Sponsor Contributor

contribute.md にくらいはファイル名の追加に関する規約を設けるべきかなとかんじました

@m1sk9
Copy link
Contributor

m1sk9 commented Jul 7, 2022

現在においてのアーキテクチャ的な問題などによるリリースの支障ではないためP0を解除

@m2en 現状では非Windowsユーザー (例: ext4をデータパーティションとして使っているLinuxユーザー) が気をつければいいという話にとどまっているのでpriority:Criticalは不適当だと考える。
@KisaragiEffective

Lowに引き下げた

@m1sk9
Copy link
Contributor

m1sk9 commented Jul 7, 2022

contribute.md にくらいはファイル名の追加に関する規約を設けるべきかなとかんじました
@raiga0310

#185 とはあまり関係がないので、 #310

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
3 participants