Skip to content

Commit

Permalink
ci(.github/workflows/ci.yaml): update CI workflow to install necessar…
Browse files Browse the repository at this point in the history
…y tools

- Removed installation of Poetry as it is already included in the Python setup
- Replaced the task of installing Task with a more generic "Install Tools" step
- Updated the configuration to install both Poetry and Task
- Removed the installation of GNU Softwares as it is no longer needed

This commit updates the CI workflow to streamline the installation process by removing redundant steps and ensuring the necessary tools are installed for the build process.
  • Loading branch information
liblaf committed Feb 4, 2024
1 parent 839da7e commit 8df5a7a
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 31 deletions.
21 changes: 6 additions & 15 deletions .github/workflows/ci.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -13,17 +13,15 @@ jobs:
steps:
- name: Checkout
uses: actions/checkout@v4
- name: Install Poetry
run: pipx install poetry
- name: Setup Python
uses: actions/setup-python@v5
with:
python-version: ${{ matrix.python-version }}
cache: poetry
- name: Install Task
uses: arduino/setup-task@v1
- name: Install Tools
uses: liblaf/template/.github/actions/install@main
with:
repo-token: ${{ github.token }}
other: poetry task
- name: Install Dependencies
run: poetry install
- name: Build Package
Expand All @@ -44,24 +42,17 @@ jobs:
steps:
- name: Checkout
uses: actions/checkout@v4
- name: Install Poetry
run: pipx install poetry
- name: Setup Python
uses: actions/setup-python@v5
with:
python-version: ${{ matrix.python-version }}
cache: poetry
- name: Install Task
uses: arduino/setup-task@v1
- name: Install Tools
uses: liblaf/template/.github/actions/install@main
with:
repo-token: ${{ github.token }}
other: poetry task
- name: Install Dependencies
run: poetry install
- name: Install GNU Softwares
uses: liblaf/template/.github/actions/install@main
with:
brew: coreutils
choco: gnuwin32-coreutils.install
- name: Build Executable
run: poetry run task dist
- name: Upload Artifacts
Expand Down
23 changes: 7 additions & 16 deletions .github/workflows/release.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -34,21 +34,19 @@ jobs:
steps:
- name: Checkout
uses: actions/checkout@v4
- name: Install Poetry
run: pipx install poetry
- name: Setup Python
uses: actions/setup-python@v5
with:
python-version: ${{ matrix.python-version }}
cache: poetry
- name: Install Task
uses: arduino/setup-task@v1
- name: Install Tools
uses: liblaf/template/.github/actions/install@main
with:
repo-token: ${{ github.token }}
other: poetry task
- name: Install Dependencies
run: poetry install
- name: Build Package
run: task build
run: poetry run task build
- name: Upload Artifacts
uses: actions/upload-artifact@v4
with:
Expand All @@ -68,24 +66,17 @@ jobs:
steps:
- name: Checkout
uses: actions/checkout@v4
- name: Install Poetry
run: pipx install poetry
- name: Setup Python
uses: actions/setup-python@v5
with:
python-version: ${{ matrix.python-version }}
cache: poetry
- name: Install Task
uses: arduino/setup-task@v1
- name: Install Tools
uses: liblaf/template/.github/actions/install@main
with:
repo-token: ${{ github.token }}
other: poetry task
- name: Install Dependencies
run: poetry install
- name: Install GNU Softwares
uses: liblaf/template/.github/actions/install@main
with:
brew: coreutils
choco: gnuwin32-coreutils.install
- name: Build Executable
run: poetry run task dist
- name: Upload Artifacts
Expand Down

0 comments on commit 8df5a7a

Please sign in to comment.