Fix #11. default oscript version (#12)

autumn-library · Jan 19, 2024 · f29a25a · f29a25a
1 parent 2d845d6
commit f29a25a
Show file tree

Hide file tree

Showing 4 changed files with 20 additions and 29 deletions.
diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml
@@ -10,7 +10,7 @@ on:
       oscript_version:
         required: false
         type: string
-        default: ""
+        default: "default"
       package_mask:
         required: false
         type: string
@@ -35,15 +35,12 @@ jobs:
         id: extract_oscript_version
         run: |
           set -ex
-          extracted_version=$(cat packagedef | grep ".ВерсияСреды(" | sed 's|[^"]*"||' | sed -r 's/".+//' || true)
-          if [ -n '${{ inputs.oscript_version }}' ]
+          if [ "${{ inputs.oscript_version }}" = "default" ]
           then
-            version='${{ inputs.oscript_version }}'
-          elif [ -n "$extracted_version" ]
-          then
-            version="$extracted_version"
+            extracted_version=$(cat packagedef | grep ".ВерсияСреды(" | sed 's|[^"]*"||' | sed -r 's/".+//' || true)
+            version="${extracted_version:-stable}"
           else
-            version='stable'
+            version="${{ inputs.oscript_version }}"
           fi
           echo "version=$version" >> $GITHUB_OUTPUT
       - name: Установка OneScript

diff --git a/.github/workflows/sonar.yml b/.github/workflows/sonar.yml
@@ -8,7 +8,7 @@ on:
         type: string
       oscript_version:
         required: false
-        default: ''
+        default: 'default'
         type: string
       sonar_host_url:
         required: false
@@ -41,15 +41,12 @@ jobs:
         id: extract_oscript_version
         run: |
           set -ex
-          extracted_version=$(cat packagedef | grep ".ВерсияСреды(" | sed 's|[^"]*"||' | sed -r 's/".+//' || true)
-          if [ -n '${{ inputs.oscript_version }}' ]
+          if [ "${{ inputs.oscript_version }}" = "default" ]
           then
-            version='${{ inputs.oscript_version }}'
-          elif [ -n "$extracted_version" ]
-          then
-            version="$extracted_version"
+            extracted_version=$(cat packagedef | grep ".ВерсияСреды(" | sed 's|[^"]*"||' | sed -r 's/".+//' || true)
+            version="${extracted_version:-stable}"
           else
-            version='stable'
+            version="${{ inputs.oscript_version }}"
           fi
           echo "version=$version" >> $GITHUB_OUTPUT
 

diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -5,7 +5,7 @@ on:
     inputs:
       oscript_version:
         required: false
-        default: ""
+        default: "default"
         type: string
       test_script_path:
         required: false
@@ -37,15 +37,12 @@ jobs:
         id: extract_oscript_version
         run: |
           set -ex
-          extracted_version=$(cat packagedef | grep ".ВерсияСреды(" | sed 's|[^"]*"||' | sed -r 's/".+//' || true)
-          if [ -n '${{ inputs.oscript_version }}' ]
+          if [ "${{ inputs.oscript_version }}" = "default" ]
           then
-            version='${{ inputs.oscript_version }}'
-          elif [ -n "$extracted_version" ]
-          then
-            version="$extracted_version"
+            extracted_version=$(cat packagedef | grep ".ВерсияСреды(" | sed 's|[^"]*"||' | sed -r 's/".+//' || true)
+            version="${extracted_version:-stable}"
           else
-            version='stable'
+            version="${{ inputs.oscript_version }}"
           fi
           echo "version=$version" >> $GITHUB_OUTPUT
 

diff --git a/README.md b/README.md
@@ -32,7 +32,7 @@ lib.system=../oscript_modules
 
 | Имя параметра    | Описание                                                                  | Значение по умолчанию                                                                   |
 | ---------------- | ------------------------------------------------------------------------- | --------------------------------------------------------------------------------------- |
-| oscript_version  | Версия движка в формате алиаса для https://github.com/oscript-library/ovm | Значение параметра метода ВерсияСреды в packagedef или `stable` в случае его отсутствия |
+| oscript_version  | Версия движка в формате алиаса для https://github.com/oscript-library/ovm. Если имеет значение `default`, производится попытка вычисления версии среды на основании вызова метода ВерсияСреды() в packagedef. Если вычислить версию не получается, используется версия stable. | default |
 | test_script_path | Путь к скрипту запуска тестов                                             | ./tasks/test.os                                                                         |
 
 ### Использование
@@ -79,13 +79,13 @@ jobs:
     strategy:
       fail-fast: false
       matrix:
-        oscript_version: ['1.8.4', 'stable', 'dev']
+        oscript_version: ['default', 'stable', 'dev']
     uses: autumn-library/workflows/.github/workflows/test.yml@v1
     with:
       oscript_version: ${{ matrix.oscript_version }}
 ```
 
-Данный пример запустит задачу на трех операционных системах с тремя разными версиями oscript - 1.8.4, последней релизной версии и последней ночной сборке.
+Данный пример запустит задачу на трех операционных системах с тремя разными версиями oscript - 1.8.4 (так как она указана в packagedef), последней релизной версии и последней ночной сборке.
 
 ## Контроль качества (SonarQube)
 
@@ -98,7 +98,7 @@ jobs:
 | Имя параметра         | Описание                                                                                                | Значение по умолчанию                                                                   |
 | --------------------- | ------------------------------------------------------------------------------------------------------- | --------------------------------------------------------------------------------------- |
 | **github_repository** | Репозиторий проекта в GitHub, для которого будет выполняться анализ, в формате "имя_владельца/название" |                                                                                         |
-| oscript_version       | Версия движка в формате алиаса для https://github.com/oscript-library/ovm                               | Значение параметра метода ВерсияСреды в packagedef или `stable` в случае его отсутствия |
+| oscript_version       | Версия движка в формате алиаса для https://github.com/oscript-library/ovm. Если имеет значение `default`, производится попытка вычисления версии среды на основании вызова метода ВерсияСреды() в packagedef. Если вычислить версию не получается, используется версия stable. | default |
 | test_script_path      | Путь к скрипту запуска тестов                                                                           | ./tasks/coverage.os                                                                     |
 | sonar_host_url        | URL сервера SonarQube                                                                                   | https://sonar.openbsl.ru                                                                |
 
@@ -139,7 +139,7 @@ jobs:
 
 | Имя параметра         | Описание                                                                                                | Значение по умолчанию                                                                   |
 | --------------------- | ------------------------------------------------------------------------------------------------------- | --------------------------------------------------------------------------------------- |
-| oscript_version       | Версия движка в формате алиаса для https://github.com/oscript-library/ovm                               | Значение параметра метода ВерсияСреды в packagedef или `stable` в случае его отсутствия |
+| oscript_version       | Версия движка в формате алиаса для https://github.com/oscript-library/ovm. Если имеет значение `default`, производится попытка вычисления версии среды на основании вызова метода ВерсияСреды() в packagedef. Если вычислить версию не получается, используется версия stable. | default |
 | package_mask      | Файловая маска собранного пакета. Не смотря на необязательность параметра, рекомендуется его передавать до исправления ошибки в шаге публикации в хаб
 | *.ospx                                                                   |