idea-fasoc/OpenFASOC

Segregate Test Workflows by Paths

Closed this issue · 4 comments

Currently, all testing workflows are run on creating a pull request.

on:
push:
branches:
- main
pull_request:
workflow_dispatch:

This is both inefficient and unnecessary. It would be better if paths are specified in the workflow events so that only the affected generators are tested in a pull request.

@harshkhandeparkar how do you know which generators are affected because of changes?

@harshkhandeparkar how do you know which generators are affected because of changes?

Some paths would affect all generators - common, for example. But generators should not interfere with each other. So we could run the tests when any common path is changed or any file under the generator's directory is changed.

sounds good. Can you modify the yml files accordingly?

sounds good. Can you modify the yml files accordingly?

Sure. I'll open a PR soon.