Skip to content
This repository has been archived by the owner on Dec 26, 2023. It is now read-only.

Commit

Permalink
fixup!
Browse files Browse the repository at this point in the history
  • Loading branch information
brokenpip3 committed Aug 14, 2023
1 parent 308ca2d commit 079bbd9
Show file tree
Hide file tree
Showing 2 changed files with 18 additions and 8 deletions.
14 changes: 12 additions & 2 deletions README.md
Original file line number Diff line number Diff line change
Expand Up @@ -63,6 +63,16 @@ For example, if you want to install `bats-support` in the `./test/bats-support`
| file-path | `/usr/lib/bats-file` | false | Bats-file path |
| file-clean | `true` | false | Bats-file: clean temp files |

## Outputs

| Key | Description |
|------------------|------------------------------------------------|
| bats-installed | True/False if bats has been installed |
| support-installed| True/False if bats-support has been installed |
| assert-installed | True/False if bats-asssert has been installed |
| detik-installed | True/False if bats-detik has been installed |
| file-installed | True/False if bats-file has been installed |

## TODO

* [X] Add more tests
Expand All @@ -73,6 +83,6 @@ For example, if you want to install `bats-support` in the `./test/bats-support`

* [X] Better Readme

* [ ] Remove sudo in case of bats libs installed in $HOME
* [x] Remove sudo in case of bats libs installed in $HOME

* [ ] Add cache for bats libs
* [x] Add cache for bats libs
12 changes: 6 additions & 6 deletions action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -84,19 +84,19 @@ inputs:
outputs:
bats-installed:
description: "True/False if bats has been installed"
value: ${{ steps.bats-install.outputs.installed }}
value: ${{ (steps.bats-install.outputs.installed == 'true') }}
support-installed:
description: "True/False if bats-support has been installed"
value: ${{ steps.support-install.outputs.installed }}
value: ${{ (steps.support-install.outputs.installed == 'true') }}
assert-installed:
description: "True/False if bats-assert has been installed"
value: ${{ steps.assert-install.outputs.installed }}
value: ${{ (steps.assert-install.outputs.installed == 'true') }}
detik-installed:
description: "True/False if bats-detik has been installed"
value: ${{ steps.detik-install.outputs.installed }}
value: ${{ (steps.detik-install.outputs.installed == 'true') }}
file-installed:
description: "True/False if bats-file has been installed"
value: ${{ steps.file-install.outputs.installed }}
value: ${{ (steps.file-install.outputs.installed == 'true') }}

runs:
using: composite
Expand All @@ -117,7 +117,7 @@ runs:
id: bats-install
shell: bash
run: |
# In $HOME to avoid sudo requirements that eventually will be removed for the library as well
# In $HOME to avoid sudo requirements
VERSION=${{ inputs.bats-version }}
DESTDIR="$HOME/.local/bin"
LIBDIR="$HOME/.local/share/bats"
Expand Down

0 comments on commit 079bbd9

Please sign in to comment.