Introduce a proper CI check for building and testing PRs

eclipse-thingweb / td-tools

Tooling for Thing Descriptions and Thing Models

https://thingweb.io

Other

3 stars 3 forks source link

Introduce a proper CI check for building and testing PRs #16

Closed danielpeintner closed 2 weeks ago

danielpeintner commented 2 months ago

As demonstrated with https://github.com/eclipse-thingweb/td-tools/pull/15 there is no CI check that shows test or build failures.

egekorkan commented 2 months ago

Given that we have different and independent tools, I propose to have separate ci files per tool and start only the relevant one based on the changes. Any thoughts?

danielpeintner commented 2 months ago

I thought about the same... the only problem I saw is how to detect best which CI setting to execute for a given PR. OR do we run any CI no matter which code gets changed (easier solution I guess)?

egekorkan commented 2 months ago

We should be able to do it folder based on paths option https://docs.github.com/en/actions/using-workflows/workflow-syntax-for-github-actions#example-including-paths