Skip to content

Update version of action tools#3493

Merged
prekucki merged 4 commits intomasterfrom
scx1332/update_actions
Mar 26, 2026
Merged

Update version of action tools#3493
prekucki merged 4 commits intomasterfrom
scx1332/update_actions

Conversation

@scx1332
Copy link
Copy Markdown
Member

@scx1332 scx1332 commented Oct 14, 2025

No description provided.

@scx1332 scx1332 requested a review from a team as a code owner October 14, 2025 13:21
@prekucki prekucki requested a review from Copilot March 26, 2026 09:22
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates the GitHub Actions integration-test workflow toolchain versions to newer Python/Poetry and a newer actions/setup-python major.

Changes:

  • Bump actions/setup-python from v5 to v6 in integration-test jobs.
  • Update the configured Python version from 3.12 to 3.14.
  • Update the pinned Poetry installer version (in multi-payment-test) from 1.8.2 to 2.2.1.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/integration-test.yml Outdated
Comment on lines +119 to +120
- name: Install Poetry
run: curl -sSL https://install.python-poetry.org | python3 - --version 1.8.2
run: curl -sSL https://install.python-poetry.org | python3 - --version 2.2.1
Copy link

Copilot AI Mar 26, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

In multi-payment-test, Poetry is installed but never used afterward (the job runs python payment_test.py directly and doesn’t call poetry). This adds an unnecessary network/download step and increases CI runtime/flakiness; consider removing this step or switching the test to run inside a Poetry-managed environment if dependencies are intended.

Copilot uses AI. Check for mistakes.
Comment thread .github/workflows/integration-test.yml Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@prekucki prekucki merged commit 21e7e39 into master Mar 26, 2026
22 of 36 checks passed
@prekucki prekucki deleted the scx1332/update_actions branch March 26, 2026 10:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants