Conversation
|
Thank you for contributing to the Autoware project! 🚧 If your pull request is in progress, switch it to draft mode. Please ensure:
|
Signed-off-by: Taeseung Sohn <taeseung.sohn@tier4.jp>
paulsohn
force-pushed
the
sohn/pre-commit-ansible-autoupdate-correct-path
branch
from
c8f3eaf to
b6c473e
Compare
Signed-off-by: Taeseung Sohn <taeseung.sohn@tier4.jp>
paulsohn
force-pushed
the
sohn/pre-commit-ansible-autoupdate-correct-path
branch
from
b934ef7 to
c21a44c
Compare
paulsohn
changed the title
Signed-off-by: Taeseung Sohn <taeseung.sohn@tier4.jp>
paulsohn
force-pushed
the
sohn/pre-commit-ansible-autoupdate-correct-path
branch
from
011e31e to
f8f32e9
Compare
| ci: | ||
| autofix_commit_msg: "style(pre-commit-ansible): autofix" | ||
| autoupdate_schedule: quarterly | ||
| autoupdate_commit_msg: "ci(pre-commit-ansible): quarterly autoupdate" | ||
|
|
There was a problem hiding this comment.
This part is unnecessary because this file is not seen by the https://github.com/apps/pre-commit-ci bot. It only sees and attempts to update the default config file.
There was a problem hiding this comment.
pre-commit-optional also has this. Should I remove it from pre-commit-optional as well?
There was a problem hiding this comment.
I'll only handle this file. If you think ci field is unnecessary for pre-commit-optional, then you can make another PR.
There was a problem hiding this comment.
Sure
I was not aware of this reply :(
There was a problem hiding this comment.
It's alright, not a high priority anyways
Signed-off-by: Taeseung Sohn <taeseung.sohn@tier4.jp>
paulsohn
force-pushed
the
sohn/pre-commit-ansible-autoupdate-correct-path
branch
from
8f57841 to
8de8946
Compare
2 participants
Description
https://github.com/autowarefoundation/sync-file-templates/actions/workflows/pre-commit-ansible-autoupdate.yaml fails because it finds
.pre-commit-config-ansible.yaml(as from directly pasted from autoware repo), while the config file to update issources/.pre-commit-config-ansible.yaml.make autoupdate quaterly for pre-commit-ansible.
How was this PR tested?
Provide correct path: https://github.com/autowarefoundation/sync-file-templates/actions/runs/22046816805 (#65 generated)
quaterly autoupdate: None. I just copy-pasted cron from other autoupdates.
Notes for reviewers
None.
Effects on system behavior
None.