pvlib-python icon indicating copy to clipboard operation
pvlib-python copied to clipboard

GitHub Actions workflow definitions could be combined

Open kandersolar opened this issue 3 years ago • 0 comments

Is your feature request related to a problem? Please describe. #1306 added GitHub Actions workflow definition files for running our test suite. That approach used separate workflow files for the "core" and "remote-data" tests because they are triggered differently. However, the two workflows have a lot in common, so it's a bit of a shame that the shared parts have to be repeated in the two files. For example when we inevitably want to add python 3.10 to the testing matrix, we'll have to edit two files instead of just one.

Describe the solution you'd like As suggested by @wholmgren in https://github.com/pvlib/pvlib-python/pull/1306#issuecomment-1155431067, it might be possible to partially combine the two workflow definitions into one by using GitHub Actions' "reusable workflow" feature: https://docs.github.com/en/actions/using-workflows/reusing-workflows

Describe alternatives you've considered N/A

Additional context N/A

kandersolar avatar Jun 15 '22 15:06 kandersolar