Tim 'mithro' Ansell
Tim 'mithro' Ansell
Please send a pull request?
Caused by https://github.com/mithro/actions-includes/blob/main/actions_includes/expressions.py#L898 GitHubmithro/actions-includesAllows including an action inside another action (by preprocessing the Yaml file). This is how composite actions should have worked. - mithro/actions-includes
@iacobfred Your issue with `hashFiles` should be fixed by https://github.com/mithro/actions-includes/commit/1813ae1d6de8302cc0584a46c77cdc0662a1aeac - I was successfully able to expand your `integration.yml` file.
Do you have an example of the string that is failing?
It looks like there might be an interaction with `||` and a `failure()`?
I was able to replicate this, ``` 1356 >>> eval("${{ steps.venv-cache.outputs.cache-hit != 'true' || failure() }}", {}) UNEXPECTED EXCEPTION: AssertionError([Value(failure), []]) Traceback (most recent call last): File "/usr/lib/python3.9/doctest.py", line 1336,...
Actually, it just seems to be the fact that I forgot to add the `failure()` function....
So, I fixed the missing `failure()` function in https://github.com/mithro/actions-includes/commit/5233fab56bd2b4ba70eac7a39e2d2b3e154b0db1 and pushed it. It turns out there is *also* an order of operation bug that I need to investigate, but I...
Any idea on what could be a workable syntax?
## Option 1 ```yml uses: nick-invision/retry@v2 with: timeout_minutes: 10 max_attempts: 3 includes-script: command: script.py ``` should expand to ```yml uses: nick-invision/retry@v2 with: timeout_minutes: 10 max_attempts: 3 shell: python command: |...