-
-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
CI: general restructure #7407
CI: general restructure #7407
Conversation
b521f1a
to
108e984
Compare
7365628
to
9d436be
Compare
b5fe0be
to
538d8bb
Compare
shell: bash | ||
run: echo "dir=$(composer config cache-dir)" >> $GITHUB_OUTPUT | ||
|
||
- name: Cache dependencies |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
so...
we are getting this error (only on the PR, i do not see it on master branch)
https://github.com/keradus/PHP-CS-Fixer/actions/runs/6748717442/job/18347647031#step:31:3
I think the cache is not stored (or maybe we need to have this config in master already?)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yep, it should store even without merge
ref https://github.com/PHP-CS-Fixer/PHP-CS-Fixer/actions/runs/6748899599/job/18348194309?pr=7413
I think we can review the whole PR already, |
2940c11
to
62d087c
Compare
…s for ci.yml and sca.yml
No description provided.