autowarefoundation / autoware.core

Apache License 2.0
45 stars 27 forks source link

chore: sync issue and PR templates #4

Closed kenji-miyake closed 2 years ago

kenji-miyake commented 2 years ago

Sync files that will be added in https://github.com/autowarefoundation/autoware/pull/56.

kenji-miyake commented 2 years ago

pre-commit.ci run

kenji-miyake commented 2 years ago

@xmfcx cc @mitsudome-r I'll explain about this CI here.

If we change the settings like this, image

and run the workflow sync-files,

image

image

this kind of PR is created. It enables us to sync files between repositories easily. https://github.com/autowarefoundation/autoware.core/pull/12

kenji-miyake commented 2 years ago

This time I've run it manually, the workflow usually runs automatically every day.

xmfcx commented 2 years ago

This time I've run it manually, the workflow usually runs automatically every day.

@kenji-miyake Why not run it with every commit?

kenji-miyake commented 2 years ago

Every commit in which repository? :thinking: Anyway, we don't have to run this workflow so frequently. I believe daily execution is enough. If necessary, you can run it anytime you like using workflow_dispatch. (What I've used this time.)

xmfcx commented 2 years ago

Every commit in which repository? 🤔 Anyway, we don't have to run this workflow so frequently. I believe daily execution is enough. If necessary, you can run it anytime you like using workflow_dispatch. (What I've used this time.)

Ah this repository doesn't have ability to get event notifications from the https://github.com/autowarefoundation/autoware-github-actions/tree/main/sync-files so we cannot trigger this when changes to that repo occurs right?

kenji-miyake commented 2 years ago

@xmfcx Actually, it's technically possible, for example by https://github.com/peter-evans/repository-dispatch. But I think it's a bit complex for this use case.