search

zap-zsh / zap

:zap: Zap is a minimal zsh plugin manager
https://www.zapzsh.com/
GNU General Public License v3.0
942 stars 33 forks source link

Add CI matrix testing #161

Closed 5c077m4n closed 1 year ago

5c077m4n commented 1 year ago

This PR includes a simple refactor to the test suite to use gh actions' matrix to parallelize and reduce a couple of lines of code

wintermi commented 1 year ago

There are two reasons for using the wintermi/zsh-lsd plugin as part of the testing script.

  1. We need to test with a plugin that is not already being loaded as part of the default-zshrc
  2. The ERROR should really be a WARNING message, but adds additional proof the plugin has been sourced.

I will update the plugin to say WARNING instead.

wintermi commented 1 year ago

Unsure if this is an issue with GitHub or the tests, the tests regularly fail, with nothing todo with the code commited and simply rerunning the failed jobs suceeds.

IMO, making the tests more robust and not likely to cause GitHub to fail should be the priority.