fix: only default to -U if there is not a version in the config […

…APE-1180] (#26)

.github/workflows/test.yaml → .github/workflows/test_ape_version.yaml

@@ -1,4 +1,4 @@
-name: tests
+name: Ape Version Test
 
 on:
   push:
@@ -9,7 +9,7 @@ on:
     paths-ignore: ['**.md']
 
 concurrency:
-  group: ${{ github.ref }}
+  group: ${{ github.ref }}-version
   cancel-in-progress: true
 
 jobs:
@@ -19,7 +19,13 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        version: ["default", "0.6.11", "==0.6.11", "git+https://github.com/ApeWorX/ape.git@main"]
+        version:
+          [
+            'default',
+            '0.6.11',
+            '==0.6.11',
+            'git+https://github.com/ApeWorX/ape.git@main',
+          ]
     steps:
       - uses: actions/checkout@v3
 

.github/workflows/test_plugins.yaml

@@ -0,0 +1,39 @@
+name: Ape Plugins Test
+
+on:
+  push:
+    branches: [main]
+    tags-ignore: ['**']
+    paths-ignore: ['**.md']
+  pull_request:
+    paths-ignore: ['**.md']
+
+concurrency:
+  group: ${{ github.ref }}-plugins
+  cancel-in-progress: true
+
+jobs:
+  run-this-action:
+    name: Run action (${{ matrix.plugins }})
+    runs-on: [ubuntu-latest]
+    strategy:
+      fail-fast: false
+      matrix:
+        plugins: ['default', 'tokens', 'tokens==0.6.1']
+    steps:
+      - uses: actions/checkout@v3
+
+      - name: Check plugins
+        id: check-plugins
+        run: |
+          if [[ "${{ matrix.plugins }}" != "default" ]]; then
+            echo "ape-plugins=${{ matrix.plugins }}" >> $GITHUB_OUTPUT
+          fi
+
+      - name: Run ape action
+        id: ape-action
+        uses: ./
+        with:
+          ape-plugins-list: ${{ steps.check-plugins.outputs.ape-plugins }}
+
+      - run: ape plugins list

README.md

@@ -17,17 +17,19 @@ Can also use a `git+` value to install a branch, commit, or tag.
 
 Example values:
 
-* `1.0.0`
-* `==1.0.0`
-* `>=1.0.0,<2.0`
-* `git+https://github.com/your-github/ape.git@your-branch`
+- `1.0.0`
+- `==1.0.0`
+- `>=1.0.0,<2.0`
+- `git+https://github.com/your-github/ape.git@your-branch`
 
 ### `ape-plugins-list`
 
 **Optional** Space-separated list of plugins to install.
-Default is to install from local `ape-config.yaml`.
+The default is to install from your project's local `ape-config.yaml`.
+If you do not have any plugin version constraints specified in your `ape-config.yaml` file, the default includes the `-U` (`--upgrade`) flag to ensure you get the latest plugin versons.
+Otherwise, it relies on the constraints you have configured.
 
-Note: When requesting a pin, put it all together like `'plugin-a plugin-b==1.2.3 plugin-c>1.2'`
+To request specific versions of plugins, use a space-separated value like this `'plugin-a plugin-b==1.2.3 plugin-c>1.2'`.
 
 ## Outputs
 

action.yml

@@ -16,7 +16,7 @@ inputs:
   ape-plugins-list:
     description: 'Space seperated list of plugins to install with relevant pinning applied'
     required: False
-    default: '-U .'
+    default: ''
 
 outputs:
   ape-version:
@@ -88,7 +88,29 @@ runs:
         fi
       shell: bash
 
-    - run: ape plugins install ${{ inputs.ape-plugins-list }}
+    - name: Check ape-config.yaml exists
+      id: check-ape-config-yaml
+      uses: andstor/file-existence-action@v2
+      with:
+        files: 'ape-config.yaml'
+
+    - run: |
+        if [[ "${{ steps.check-ape-config-yaml.outputs.files_exists }}"  == "true" ]]; then
+          version_present=$(sed -n '/^plugins:/,/^[^ ]/{/version:/p;}' "ape-config.yaml" | grep -c version)
+        else
+          version_present=0
+        fi
+
+        if [[ "${version_present}" == "1" ]] && [[ -z "${{ inputs.ape-plugins-list }}" ]]; then
+          plugins_value="."
+        elif [[ -z "${{ inputs.ape-plugins-list }}" ]]; then
+          plugins_value="--upgrade ."
+        else
+          plugins_value="${{ inputs.ape-plugins-list }}"
+        fi
+
+        ape plugins install "$plugins_value"
+
       shell: bash
 
     - name: Find if requirements.txt if exists

ape-config.yaml

@@ -0,0 +1,4 @@
+# This file exists only as a test for the action.
+plugins:
+  - name: tokens
+    version: 0.6.1

mdformat.yaml

@@ -0,0 +1 @@
+number = true