Skip to content

add dependency-update workflow template (fixes #683)#756

Open
Rimsha2535 wants to merge 12 commits intomainfrom
feature/dependency-update-workflow
Open

add dependency-update workflow template (fixes #683)#756
Rimsha2535 wants to merge 12 commits intomainfrom
feature/dependency-update-workflow

Conversation

@Rimsha2535
Copy link
Copy Markdown

@Rimsha2535 Rimsha2535 commented Mar 19, 2026

Fixes #683

Checklist

Note: If any of the items in the checklist are not relevant to your PR, just check the box.

For any Pull Request

Is the following correct:

  • the title of the Pull Request?
  • the title of the corresponding issue?
  • there are no other open [Pull Requests](../../../../pulls) for the same update/change?
  • that the issue which this Pull Request fixes ("Fixes...") is mentioned?

When Changes Were Made

Did you:

  • update the changelog?
  • update the cookiecutter-template?
  • update the implementation?
  • check coverage and add tests: unit tests and, if relevant, integration tests?
  • update the User Guide & other documentation?
  • resolve any failing CI criteria (incl. Sonar quality gate)?

When Preparing a Release

Have you:

  • thought about version number (major, minor, patch)?
  • checked Exasol packages for updates and resolved open vulnerabilities, if easily possible?

Notes

  • Changelog was not updated because this is an internal workflow/template change.
  • No separate cookiecutter-template update was needed because the workflow template itself was updated.
  • CI checks are currently failing and will be fixed.

@Rimsha2535 Rimsha2535 requested a review from kratz00 March 19, 2026 14:30
@kratz00 kratz00 changed the title add dependency-update workflow template add dependency-update workflow template (fixes #683) Mar 19, 2026
@Rimsha2535 Rimsha2535 requested a review from ArBridgeman March 30, 2026 12:41
Comment thread .github/workflows/dependency-update.yml
Comment thread exasol/toolbox/templates/github/workflows/dependency-update.yml
Comment thread exasol/toolbox/templates/github/workflows/dependency-update.yml Outdated
Comment thread exasol/toolbox/templates/github/workflows/dependency-update.yml Outdated
Comment thread .github/workflows/dependency-update.yml
Comment thread doc/github_actions/dependency_update.rst Outdated
Comment thread doc/github_actions/dependency_update.rst Outdated
Comment thread exasol/toolbox/templates/github/workflows/dependency-update.yml Outdated
Comment thread test/integration/project-template/nox_test.py
Comment thread exasol/toolbox/templates/github/workflows/dependency-update.yml Outdated
Comment thread exasol/toolbox/templates/github/workflows/dependency-update.yml Outdated
Comment thread exasol/toolbox/templates/github/workflows/dependency-update.yml Outdated
@Rimsha2535 Rimsha2535 requested a review from ArBridgeman April 14, 2026 14:35
Comment thread doc/user_guide/features/github_workflows/index.rst Outdated
Comment thread doc/user_guide/features/github_workflows/index.rst Outdated
Comment thread doc/user_guide/features/github_workflows/index.rst Outdated
Comment thread doc/user_guide/features/github_workflows/index.rst Outdated
Comment thread doc/user_guide/features/github_workflows/index.rst
Comment thread .github/workflows/dependency-update.yml
@ArBridgeman
Copy link
Copy Markdown
Collaborator

ArBridgeman commented Apr 15, 2026

The GitHub workflow code looked good, but it's always good test, so here are those tests done via
another branch (nearly identical to this one):

Use cases:

  1. ✅ No vulnerability detected, so no update
    https://github.com/exasol/python-toolbox/actions/runs/24443525540/job/71414050584
  2. ✅ Vulnerability detected, do an update, & create PR
    https://github.com/exasol/python-toolbox/actions/runs/24444147768/job/71416082184

Example PR:
#780

Like @ckunki said, we likely need to modify this text more to tell the user what to do. But it sounds like this would be done in a later effort.

Comment thread exasol/toolbox/templates/github/workflows/dependency-update.yml Outdated
Comment thread .github/workflows/dependency-update.yml Outdated
Comment thread .github/workflows/dependency-update.yml
Copy link
Copy Markdown
Contributor

@ckunki ckunki left a comment

Choose a reason for hiding this comment

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

Added 2 comments

Comment thread test/integration/tools/workflow_integration_test.py Outdated
Comment thread .github/workflows/dependency-update.yml Outdated
@ArBridgeman ArBridgeman temporarily deployed to manual-approval May 4, 2026 10:01 — with GitHub Actions Inactive
@ArBridgeman ArBridgeman marked this pull request as ready for review May 4, 2026 10:03
@ArBridgeman
Copy link
Copy Markdown
Collaborator

ArBridgeman commented May 4, 2026

The GitHub workflow code looked good, but it's always good test, so @Rimsha2535 and I did that on May-4th.
Here are those tests done via
#815

Use cases:
- ✅ Branch is not default one, so don't run it
https://github.com/exasol/python-toolbox/actions/runs/25308604711/job/74190054621
- ✅ No vulnerability detected, so no update
https://github.com/exasol/python-toolbox/actions/runs/25308753000/job/74190514343
- ✅ Vulnerability detected, do an update, create PR, & notify in Slack
https://github.com/exasol/python-toolbox/actions/runs/25312452233/job/74202272517

Example PR:
#818

I'll modify the docs & checks in another issue: #792

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented May 4, 2026

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.

Create a template workflow to execute poetry update

4 participants