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

任意のパスをビルドキャッシュから除外する custom-cache-path を追加する #3

Merged
merged 3 commits into from
Dec 6, 2021

Conversation

t2y
Copy link
Contributor

@t2y t2y commented Dec 4, 2021

説明

背景/意図

java (maven?) では -SNAPSHOT という接尾辞を使って開発中の jar のバージョンを表す。github actions のワークフローによってはビルドキャッシュで SNAPSHOT を扱うとトラブルになるケースもある。SNAPSHOT を含む任意のパスをキャッシュ対象外とすることでそういった運用の問題を解決する。

変更内容

次のように custom-cache-pathexclude パターン を記述することで任意のパスをキャッシュ対象外とする。

- name: Update library versions
  uses: HoshinoResort/hr-library-auto-update@v1
  with:
    maven-test-command: "test -DfailIfNoTests=false -Dtest='!IntegrationTest'"
    maven-settings-xml-path: "${{ github.workspace }}/settings.xml"
    custom-cache-path: '!~/.m2/repository/com/subdomain'
    push-on-success: 'true'

懸念事項

次の issue にあるように exclude パターンはユーザーの直観通りの振る舞いをするわけではなく、設定方法の組み合わせによっては意図したように除外されない場合がある。あまり要件にあわせた柔軟な設定はできない可能性がある。

リファレンス

@t2y t2y self-assigned this Dec 4, 2021
@t2y t2y linked an issue Dec 4, 2021 that may be closed by this pull request
@t2y
Copy link
Contributor Author

t2y commented Dec 6, 2021

軽微な修正なのでマージする。なにか懸念があったら後でもよいのでコメントください。

@t2y t2y merged commit a48a6a4 into develop Dec 6, 2021
@t2y t2y deleted the feat/2/add-exclude-path branch December 6, 2021 04:46
@t2y t2y changed the title refs #2 任意のパスをビルドキャッシュから除外する custom-cache-path を追加する 任意のパスをビルドキャッシュから除外する custom-cache-path を追加する Jan 26, 2022
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.

任意のパスをビルドキャッシュの対象外とする
1 participant